commit 8c9b8783e26c3fbfc6d111017796256df6ea93dd
author Eric Roman <eroman@chromium.org> Tue Sep 22 18:32:43 2015 -0700
committer Adam Langley <agl@google.com> Thu Sep 24 00:04:59 2015 +0000
tree 5a82b6d53e96a32de01e2d5f4c57be2a21fb106f
parent f0e8d537de8300ceac48d37174266490f70e00b1

Change some "int" variables to "size_t" in ssl3_handshake_mac().

BUG=https://crbug.com/535039

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

ssl/s3_enc.c

diff --git a/ssl/s3_enc.c b/ssl/s3_enc.c
index f1924c0..aa0d717 100644
--- a/ssl/s3_enc.c
+++ b/ssl/s3_enc.c
@@ -334,7 +334,7 @@
 static int ssl3_handshake_mac(SSL *s, int md_nid, const char *sender, int len,
                               uint8_t *p) {
   unsigned int ret;
-  int npad, n;
+  size_t npad, n;
   unsigned int i;
   uint8_t md_buf[EVP_MAX_MD_SIZE];
   EVP_MD_CTX ctx;
@@ -357,9 +357,6 @@
   }
 
   n = EVP_MD_CTX_size(&ctx);
-  if (n < 0) {
-    return 0;
-  }
 
   npad = (48 / n) * n;
   if (sender != NULL) {