diff options
author | Richard Levitte <levitte@openssl.org> | 2018-10-29 16:27:43 +0100 |
---|---|---|
committer | Richard Levitte <levitte@openssl.org> | 2018-10-29 19:06:25 +0100 |
commit | 88e3cf0a1024f4afaf8e44553526eb326db102bc (patch) | |
tree | 4441c22e0be02d1741d8594ab3625cddf43c8fff | |
parent | ce5d64c79c4d809ece8fe28a5b62915467a1c0e1 (diff) |
test/evp_test.c: Fixed strcmp() fault in mac_test_init()
When wanting to compare the end of a string with another string, make
sure not to start somewhere before the start of the first string.
[extended tests]
Reviewed-by: Matt Caswell <matt@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/7520)
-rw-r--r-- | test/evp_test.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/test/evp_test.c b/test/evp_test.c index 311814b1bd..25b10d3314 100644 --- a/test/evp_test.c +++ b/test/evp_test.c @@ -863,7 +863,8 @@ static int mac_test_init(EVP_TEST *t, const char *alg) size_t sz = strlen(alg); static const char epilogue[] = " by EVP_PKEY"; - if (strcmp(alg + sz - (sizeof(epilogue) - 1), epilogue) == 0) + if (sz >= sizeof(epilogue) + && strcmp(alg + sz - (sizeof(epilogue) - 1), epilogue) == 0) sz -= sizeof(epilogue) - 1; if (strncmp(alg, "HMAC", sz) == 0) { |