Fix more shadowed variable warnings
authorMatt Caswell <matt@openssl.org>
Tue, 25 Oct 2016 23:05:25 +0000 (00:05 +0100)
committerMatt Caswell <matt@openssl.org>
Fri, 28 Oct 2016 08:48:54 +0000 (09:48 +0100)
Reviewed-by: Richard Levitte <levitte@openssl.org>
crypto/bio/bio_lib.c
crypto/bio/bio_meth.c

index 430eac8bb591178bf2b1d8028f6b24928ad24687..2bd337c36248db9e7926033ea97a770fec667cb0 100644 (file)
@@ -247,7 +247,7 @@ int BIO_method_type(const BIO *b)
  * This is for compatibility with the old style BIO_read(), where existing code
  * may make assumptions about the return value that it might get.
  */
-static int bio_read_intern(BIO *b, void *data, size_t dlen, size_t *read)
+static int bio_read_intern(BIO *b, void *data, size_t dlen, size_t *readbytes)
 {
     int ret;
 
@@ -258,7 +258,7 @@ static int bio_read_intern(BIO *b, void *data, size_t dlen, size_t *read)
 
     if ((b->callback != NULL || b->callback_ex != NULL) &&
         ((ret = (int)bio_call_callback(b, BIO_CB_READ, data, dlen, 0, 0L, 1L,
-                                       read)) <= 0))
+                                       readbytes)) <= 0))
         return ret;
 
     if (!b->init) {
@@ -266,17 +266,17 @@ static int bio_read_intern(BIO *b, void *data, size_t dlen, size_t *read)
         return -2;
     }
 
-    ret = b->method->bread(b, data, dlen, read);
+    ret = b->method->bread(b, data, dlen, readbytes);
 
     if (ret > 0)
         b->num_read += (uint64_t)*read;
 
     if (b->callback != NULL || b->callback_ex != NULL)
         ret = (int)bio_call_callback(b, BIO_CB_READ | BIO_CB_RETURN, data,
-                                     dlen, 0, 0L, ret, read);
+                                     dlen, 0, 0L, ret, readbytes);
 
     /* Shouldn't happen */
-    if (ret > 0 && *read > dlen) {
+    if (ret > 0 && *readbytes > dlen) {
         BIOerr(BIO_F_BIO_READ_INTERN, ERR_R_INTERNAL_ERROR);
         return -1;
     }
@@ -286,27 +286,27 @@ static int bio_read_intern(BIO *b, void *data, size_t dlen, size_t *read)
 
 int BIO_read(BIO *b, void *data, int dlen)
 {
-    size_t read;
+    size_t readbytes;
     int ret;
 
     if (dlen < 0)
         return 0;
 
-    ret = bio_read_intern(b, data, (size_t)dlen, &read);
+    ret = bio_read_intern(b, data, (size_t)dlen, &readbytes);
 
     if (ret > 0) {
-        /* *read should always be <= outl */
-        ret = (int)read;
+        /* *readbytes should always be <= outl */
+        ret = (int)readbytes;
     }
 
     return ret;
 }
 
-int BIO_read_ex(BIO *b, void *data, size_t dlen, size_t *read)
+int BIO_read_ex(BIO *b, void *data, size_t dlen, size_t *readbytes)
 {
     int ret;
 
-    ret = bio_read_intern(b, data, dlen, read);
+    ret = bio_read_intern(b, data, dlen, readbytes);
 
     if (ret > 0)
         ret = 1;
@@ -431,7 +431,7 @@ int BIO_puts(BIO *b, const char *in)
 int BIO_gets(BIO *b, char *out, int outl)
 {
     int ret;
-    size_t read = 0;
+    size_t readbytes = 0;
 
     if ((b == NULL) || (b->method == NULL) || (b->method->bgets == NULL)) {
         BIOerr(BIO_F_BIO_GETS, BIO_R_UNSUPPORTED_METHOD);
@@ -457,20 +457,20 @@ int BIO_gets(BIO *b, char *out, int outl)
     ret = b->method->bgets(b, out, outl);
 
     if (ret > 0) {
-        read = ret;
+        readbytes = ret;
         ret = 1;
     }
 
     if (b->callback != NULL || b->callback_ex != NULL)
         ret = (int)bio_call_callback(b, BIO_CB_GETS | BIO_CB_RETURN, out, outl,
-                                     0, 0L, ret, &read);
+                                     0, 0L, ret, &readbytes);
 
     if (ret > 0) {
         /* Shouldn't happen */
-        if (read > (size_t)outl)
+        if (readbytes > (size_t)outl)
             ret = -1;
         else
-            ret = (int)read;
+            ret = (int)readbytes;
     }
 
     return ret;
index 5eaa1fd461cb2b841504d754556bc64c2f506e1b..588c5170f7f546ffc6bc484fee5047236edd1529 100644 (file)
@@ -107,7 +107,7 @@ int (*BIO_meth_get_read_ex(BIO_METHOD *biom)) (BIO *, char *, size_t, size_t *)
 }
 
 /* Conversion for old style bread to new style */
-int bread_conv(BIO *bio, char *data, size_t datal, size_t *read)
+int bread_conv(BIO *bio, char *data, size_t datal, size_t *readbytes)
 {
     int ret;
 
@@ -117,11 +117,11 @@ int bread_conv(BIO *bio, char *data, size_t datal, size_t *read)
     ret = bio->method->bread_old(bio, data, (int)datal);
 
     if (ret <= 0) {
-        *read = 0;
+        *readbytes = 0;
         return ret;
     }
 
-    *read = (size_t)ret;
+    *readbytes = (size_t)ret;
 
     return 1;
 }