PR: 2138
authorDr. Stephen Henson <steve@openssl.org>
Tue, 26 Jan 2010 18:08:42 +0000 (18:08 +0000)
committerDr. Stephen Henson <steve@openssl.org>
Tue, 26 Jan 2010 18:08:42 +0000 (18:08 +0000)
Submitted by: Kevin Regan <k.regan@f5.com>

Clear stat structure if -DPURIFY is set to avoid problems on some
platforms which include unitialised fields.

crypto/rand/randfile.c

index e12fe0b90c6cc89eb59a02de2976f67a67e52ff3..84276d7d4602ea4525492901e4d0c6de9f5da7fe 100644 (file)
@@ -117,6 +117,15 @@ int RAND_load_file(const char *file, long bytes)
 
        if (file == NULL) return(0);
 
+#ifdef PURIFY
+       /* struct stat can have padding and unused fields that may not be
+        * initialized in the call to stat(). We need to clear the entire
+        * structure before calling RAND_add() to avoid complaints from
+        * applications such as Valgrind.
+        */
+       memset(&sb, 0, sizeof(sb));
+#endif
+
        if (stat(file,&sb) < 0) return(0);
        RAND_add(&sb,sizeof(sb),0.0);
        if (bytes == 0) return(ret);