From bb6582896d0b5910e694f003d6b207e63396e36f Mon Sep 17 00:00:00 2001 From: Jacob Barthelmeh Date: Thu, 15 Jun 2017 11:55:37 -0600 Subject: [PATCH] add sanity check for wolfSSL_X509_NAME_oneline function --- src/ssl.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/ssl.c b/src/ssl.c index 9c5207f45..5b93ed2e3 100755 --- a/src/ssl.c +++ b/src/ssl.c @@ -13523,7 +13523,14 @@ static void ExternalFreeX509(WOLFSSL_X509* x509) malloc buffer, call responsible for freeing */ char* wolfSSL_X509_NAME_oneline(WOLFSSL_X509_NAME* name, char* in, int sz) { - int copySz = min(sz, name->sz); + int copySz; + + if (name == NULL) { + WOLFSSL_MSG("WOLFSSL_X509_NAME pointer was NULL"); + return NULL; + } + + copySz = min(sz, name->sz); WOLFSSL_ENTER("wolfSSL_X509_NAME_oneline"); if (!name->sz) return in; @@ -13539,7 +13546,7 @@ static void ExternalFreeX509(WOLFSSL_X509* x509) #endif } - if (copySz == 0) + if (copySz <= 0) return in; XMEMCPY(in, name->name, copySz - 1);