diff options
| author | Brian Smith <brian@briansmith.org> | 2016-03-16 01:44:36 +0300 |
|---|---|---|
| committer | David Benjamin <davidben@google.com> | 2016-04-18 23:42:22 +0300 |
| commit | 69f0532c8530ef3b26a63f6fbf819bb006c13222 (patch) | |
| tree | 4688699aebdb7392d9e3b118d6e7a8900443d295 /crypto/rsa | |
| parent | 2a92031bb4dceef47b8ee49a9184d7e572fb841b (diff) | |
Use |memcmp| instead of |CRYPTO_memcmp| in |RSA_verify|.
|CRYPTO_memcmp| isn't necessary because there is no secret data being
acted on here.
Change-Id: Ib678d5d4fc16958aca409a93df139bdff8cb73fb
Reviewed-on: https://boringssl-review.googlesource.com/7465
Reviewed-by: David Benjamin <davidben@google.com>
Diffstat (limited to 'crypto/rsa')
| -rw-r--r-- | crypto/rsa/rsa.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/crypto/rsa/rsa.c b/crypto/rsa/rsa.c index daaa082f..e5f98916 100644 --- a/crypto/rsa/rsa.c +++ b/crypto/rsa/rsa.c @@ -496,7 +496,7 @@ int RSA_verify(int hash_nid, const uint8_t *msg, size_t msg_len, goto out; } - if (len != signed_msg_len || CRYPTO_memcmp(buf, signed_msg, len) != 0) { + if (len != signed_msg_len || memcmp(buf, signed_msg, len) != 0) { OPENSSL_PUT_ERROR(RSA, RSA_R_BAD_SIGNATURE); goto out; } |