From: Peng Fan Date: Wed, 23 Dec 2015 04:07:24 +0000 (+0800) Subject: common: env: initialize scalar variable X-Git-Tag: v2016.01-rc4~2 X-Git-Url: https://git.librecmc.org/?a=commitdiff_plain;h=5a6894397a657edec5d0cf4e20968cc66a368c51;p=oweals%2Fu-boot.git common: env: initialize scalar variable Before calling hsearch_r, initialize callback entry to NULL. Coverity log: " Uninitialized scalar variable (UNINIT) uninit_use_in_call: Using uninitialized value e. Field e.callback is uninitialized when calling hsearch_r. " Reported-by: Coverity Signed-off-by: Peng Fan Cc: Tom Rini Cc: Simon Glass --- diff --git a/common/env_callback.c b/common/env_callback.c index f4d3dbd77f..1957cc1996 100644 --- a/common/env_callback.c +++ b/common/env_callback.c @@ -97,6 +97,7 @@ static int set_callback(const char *name, const char *value, void *priv) e.key = name; e.data = NULL; + e.callback = NULL; hsearch_r(e, FIND, &ep, &env_htab, 0); /* does the env variable actually exist? */ diff --git a/common/env_flags.c b/common/env_flags.c index e682d85178..771935508c 100644 --- a/common/env_flags.c +++ b/common/env_flags.c @@ -455,6 +455,7 @@ static int set_flags(const char *name, const char *value, void *priv) e.key = name; e.data = NULL; + e.callback = NULL; hsearch_r(e, FIND, &ep, &env_htab, 0); /* does the env variable actually exist? */