diff options
author | Dmitry-Me <wipedout@yandex.ru> | 2016-02-29 11:55:13 +0300 |
---|---|---|
committer | Rich Salz <rsalz@openssl.org> | 2016-02-29 17:33:44 -0500 |
commit | edae9834b6eeeb7af8839187a6de82131991ec4b (patch) | |
tree | 346c8c0365748aaea613df8fa8332d686b92f77a | |
parent | 1c03c81f52c15eb4f4b8aac158313c31be82e3e5 (diff) |
GH762: Reuse strdup()
Signed-off-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Richard Levitte <levitte@openssl.org>
-rw-r--r-- | crypto/dso/dso_lib.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/crypto/dso/dso_lib.c b/crypto/dso/dso_lib.c index 0921fb2409..c410eac9ee 100644 --- a/crypto/dso/dso_lib.c +++ b/crypto/dso/dso_lib.c @@ -341,12 +341,11 @@ int DSO_set_filename(DSO *dso, const char *filename) return (0); } /* We'll duplicate filename */ - copied = OPENSSL_malloc(strlen(filename) + 1); + copied = OPENSSL_strdup(filename); if (copied == NULL) { DSOerr(DSO_F_DSO_SET_FILENAME, ERR_R_MALLOC_FAILURE); return (0); } - OPENSSL_strlcpy(copied, filename, strlen(filename) + 1); OPENSSL_free(dso->filename); dso->filename = copied; return (1); @@ -390,12 +389,11 @@ char *DSO_convert_filename(DSO *dso, const char *filename) result = dso->meth->dso_name_converter(dso, filename); } if (result == NULL) { - result = OPENSSL_malloc(strlen(filename) + 1); + result = OPENSSL_strdup(filename); if (result == NULL) { DSOerr(DSO_F_DSO_CONVERT_FILENAME, ERR_R_MALLOC_FAILURE); return (NULL); } - OPENSSL_strlcpy(result, filename, strlen(filename) + 1); } return (result); } |