summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArnaud Lacombe <lacombar@gmail.com>2010-10-13 20:43:28 -0400
committerArnaud Lacombe <lacombar@gmail.com>2011-07-02 01:04:38 -0400
commit275744cc8da0fd006e8f8fc1090f01dc0296810d (patch)
tree93a812aaa6c9d867719a8d00a960d88cc7db00fa
parent8494453ad50599b0c9d099a230e423a89222ad08 (diff)
downloadop-kernel-dev-275744cc8da0fd006e8f8fc1090f01dc0296810d.zip
op-kernel-dev-275744cc8da0fd006e8f8fc1090f01dc0296810d.tar.gz
kconfig/conf: reduce the scope of `defconfig_file'
This variable is not used outside of main() so there is not much reason keeping it global. Ensure it is initialized as gcc has no way to know that normal execution path expect only one option switch to be given on the command line (except when we request help). As a result, we always initialize `defconfig_file' before using it. Signed-off-by: Arnaud Lacombe <lacombar@gmail.com>
-rw-r--r--scripts/kconfig/conf.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/scripts/kconfig/conf.c b/scripts/kconfig/conf.c
index 08c05bc..6431f5c 100644
--- a/scripts/kconfig/conf.c
+++ b/scripts/kconfig/conf.c
@@ -34,8 +34,6 @@ enum input_mode {
oldnoconfig,
} input_mode = oldaskconfig;
-char *defconfig_file;
-
static int indent = 1;
static int valid_stdin = 1;
static int sync_kconfig;
@@ -462,7 +460,7 @@ static struct option long_opts[] = {
int main(int ac, char **av)
{
int opt;
- const char *name;
+ const char *name, *defconfig_file = NULL /* gcc uninit */;
struct stat tmpstat;
setlocale(LC_ALL, "");
OpenPOWER on IntegriCloud