diff options
| author | Adam Langley <agl@chromium.org> | 2014-09-03 01:28:49 +0400 |
|---|---|---|
| committer | Adam Langley <agl@google.com> | 2014-09-03 02:39:41 +0400 |
| commit | b2cb0ece7678586c9ca9c02dbc40069037b4f5a9 (patch) | |
| tree | e1942774ab39d4d8437301e92e30f0041ff74ed5 /crypto/ecdsa | |
| parent | ed8270a55c3845abbc85dfeed358597fef059ea9 (diff) | |
Fix minor issues found by Clang's analysis.
Thanks to Denis Denisov for running the analysis.
Change-Id: I80810261e013423e746fd8d8afefb3581cffccc0
Reviewed-on: https://boringssl-review.googlesource.com/1701
Reviewed-by: Adam Langley <agl@google.com>
Diffstat (limited to 'crypto/ecdsa')
| -rw-r--r-- | crypto/ecdsa/ecdsa.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/crypto/ecdsa/ecdsa.c b/crypto/ecdsa/ecdsa.c index 067fd6cc..ddc3e617 100644 --- a/crypto/ecdsa/ecdsa.c +++ b/crypto/ecdsa/ecdsa.c @@ -140,8 +140,9 @@ int ECDSA_do_verify(const uint8_t *digest, size_t digest_len, } /* check input values */ - if (eckey == NULL || (group = EC_KEY_get0_group(eckey)) == NULL || - (pub_key = EC_KEY_get0_public_key(eckey)) == NULL || sig == NULL) { + if ((group = EC_KEY_get0_group(eckey)) == NULL || + (pub_key = EC_KEY_get0_public_key(eckey)) == NULL || + sig == NULL) { OPENSSL_PUT_ERROR(ECDSA, ECDSA_do_verify, ECDSA_R_MISSING_PARAMETERS); return 0; } |