It isn't necessary to initialise a struct stat before a stat(2) system call.

The initialisation was also flawed, failing to account for padding and
alignment bytes.

Reviewed-by: Richard Levitte <levitte@openssl.org>
(Merged from https://github.com/openssl/openssl/pull/8611)

diff --git a/crypto/conf/conf_def.c b/crypto/conf/conf_def.c
index 594f7c5e5a32d63d27c92eb09540b08373af48a7..277e4d64c96bc42299b88c2ce9903ba512e8e7d0 100644 (file)
--- a/crypto/conf/conf_def.c
+++ b/crypto/conf/conf_def.c
@@ -669,7 +669,7 @@ static int str_copy(CONF *conf, char *section, char **pto, char *from)
 static BIO *process_include(char *include, OPENSSL_DIR_CTX **dirctx,
                             char **dirpath)
 {
-    struct stat st = { 0 };
+    struct stat st;
     BIO *next;
 
     if (stat(include, &st) < 0) {