apps/speed.c: initialize buffers
authorPatrick Steuer <patrick.steuer@de.ibm.com>
Mon, 4 Dec 2017 17:32:12 +0000 (18:32 +0100)
committerAndy Polyakov <appro@openssl.org>
Sat, 9 Dec 2017 20:44:00 +0000 (21:44 +0100)
Stop valgrind's complaints about uninitialized values.

Signed-off-by: Patrick Steuer <patrick.steuer@de.ibm.com>
Reviewed-by: Rich Salz <rsalz@openssl.org>
Reviewed-by: Andy Polyakov <appro@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/4842)

apps/speed.c

index 926778eced966e6defc487827f2093cfc07bf4cc..4c6ee081606888a6eba1fd6123ccb702804f4946 100644 (file)
@@ -1268,6 +1268,7 @@ int speed_main(int argc, char **argv)
     long count = 0;
     int size_num = OSSL_NELEM(lengths_list);
     int keylen;
+    int buflen;
 #ifndef NO_FORK
     int multi = 0;
 #endif
@@ -1605,12 +1606,12 @@ int speed_main(int argc, char **argv)
             }
         }
 
-        loopargs[i].buf_malloc =
-            app_malloc(lengths[size_num - 1] + MAX_MISALIGNMENT + 1,
-                       "input buffer");
-        loopargs[i].buf2_malloc =
-            app_malloc(lengths[size_num - 1] + MAX_MISALIGNMENT + 1,
-                       "input buffer");
+        buflen = lengths[size_num - 1] + MAX_MISALIGNMENT + 1;
+        loopargs[i].buf_malloc = app_malloc(buflen, "input buffer");
+        loopargs[i].buf2_malloc = app_malloc(buflen, "input buffer");
+        memset(loopargs[i].buf_malloc, 0, buflen);
+        memset(loopargs[i].buf2_malloc, 0, buflen);
+
         /* Align the start of buffers on a 64 byte boundary */
         loopargs[i].buf = loopargs[i].buf_malloc + misalign;
         loopargs[i].buf2 = loopargs[i].buf2_malloc + misalign;