diff options
| author | Pauli <paul.dale@oracle.com> | 2020-10-09 09:32:04 +1000 |
|---|---|---|
| committer | Pauli <paul.dale@oracle.com> | 2020-10-12 10:27:16 +1000 |
| commit | fb33f99409972ea3d217399558d3baed4a57926d (patch) | |
| tree | ec1c0c2ec596d58630384fd120fc8ac240d076aa /test | |
| parent | 19431e5e44144b57ab936ddb93fe75fe34279290 (diff) | |
coverity 1414446 out-of-bounds access: allocate \0 terminator byte to be safe
Reviewed-by: Tomas Mraz <tmraz@fedoraproject.org>
(Merged from https://github.com/openssl/openssl/pull/13091)
Diffstat (limited to 'test')
| -rw-r--r-- | test/v3nametest.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/v3nametest.c b/test/v3nametest.c index 9b81988ddd..df12c15f09 100644 --- a/test/v3nametest.c +++ b/test/v3nametest.c @@ -289,10 +289,10 @@ static int run_cert(X509 *crt, const char *nameincert, for (; *pname != NULL; ++pname) { int samename = strcasecmp(nameincert, *pname) == 0; size_t namelen = strlen(*pname); - char *name = OPENSSL_malloc(namelen); + char *name = OPENSSL_malloc(namelen + 1); int match, ret; - memcpy(name, *pname, namelen); + memcpy(name, *pname, namelen + 1); match = -1; if (!TEST_int_ge(ret = X509_check_host(crt, name, namelen, 0, NULL), |