commit 048d4ad5ee9ad58c121a0eaf65131f8fb7a39f87
author Adam Langley <agl@google.com> Fri Aug 14 13:05:21 2015 -0700
committer Adam Langley <agl@google.com> Mon Aug 17 19:12:07 2015 +0000
tree 1ae573e5913ff6e5eab90ac12c349c734583495b
parent 74279b63428d9b25052207dd81121b67a847c20e

Remove superfluous “!= 0” in if.

“!= 0” is implicit in if statements and it looks very weird here.

Change-Id: I7f4e71c479b8ff9821a040f1c542b15af19b8aed
Reviewed-on: https://boringssl-review.googlesource.com/5720
Reviewed-by: David Benjamin <davidben@chromium.org>
Reviewed-by: Adam Langley <agl@google.com>

crypto/cipher/e_aes.c

diff --git a/crypto/cipher/e_aes.c b/crypto/cipher/e_aes.c
index 24013dc..c014d19 100644
--- a/crypto/cipher/e_aes.c
+++ b/crypto/cipher/e_aes.c
@@ -709,7 +709,7 @@
   } else {
     if (!ctx->encrypt) {
       if (gctx->taglen < 0 ||
-          !CRYPTO_gcm128_finish(&gctx->gcm, ctx->buf, gctx->taglen) != 0) {
+          !CRYPTO_gcm128_finish(&gctx->gcm, ctx->buf, gctx->taglen)) {
         return -1;
       }
       gctx->iv_set = 0;