Static variables are initialized to zero by default, so remove explicit initalization. This patch fixes the checkpatch issue. Signed-off-by: Varsha Rao <rvarsha016@xxxxxxxxx> --- iptables/ip6tables-restore.c | 2 +- iptables/ip6tables-save.c | 2 +- iptables/iptables-restore.c | 2 +- iptables/iptables-save.c | 2 +- iptables/iptables-xml.c | 10 +++++----- iptables/xtables-arp.c | 2 +- 6 files changed, 10 insertions(+), 10 deletions(-) diff --git a/iptables/ip6tables-restore.c b/iptables/ip6tables-restore.c index 8a47f09..30a4ade 100644 --- a/iptables/ip6tables-restore.c +++ b/iptables/ip6tables-restore.c @@ -26,7 +26,7 @@ #define DEBUGP(x, args...) #endif -static int counters = 0, verbose = 0, noflush = 0, wait = 0; +static int counters, verbose, noflush, wait; static struct timeval wait_interval = { .tv_sec = 1, diff --git a/iptables/ip6tables-save.c b/iptables/ip6tables-save.c index 053413a..c2495d2 100644 --- a/iptables/ip6tables-save.c +++ b/iptables/ip6tables-save.c @@ -18,7 +18,7 @@ #include "ip6tables.h" #include "ip6tables-multi.h" -static int show_counters = 0; +static int show_counters; static const struct option options[] = { {.name = "counters", .has_arg = false, .val = 'c'}, diff --git a/iptables/iptables-restore.c b/iptables/iptables-restore.c index 7bb06d8..15db358 100644 --- a/iptables/iptables-restore.c +++ b/iptables/iptables-restore.c @@ -23,7 +23,7 @@ #define DEBUGP(x, args...) #endif -static int counters = 0, verbose = 0, noflush = 0, wait = 0; +static int counters, verbose, noflush, wait; static struct timeval wait_interval = { .tv_sec = 1, diff --git a/iptables/iptables-save.c b/iptables/iptables-save.c index e8ae9c6..fbc6059 100644 --- a/iptables/iptables-save.c +++ b/iptables/iptables-save.c @@ -17,7 +17,7 @@ #include "iptables.h" #include "iptables-multi.h" -static int show_counters = 0; +static int show_counters; static const struct option options[] = { {.name = "counters", .has_arg = false, .val = 'c'}, diff --git a/iptables/iptables-xml.c b/iptables/iptables-xml.c index 740a563..2e093b5 100644 --- a/iptables/iptables-xml.c +++ b/iptables/iptables-xml.c @@ -34,9 +34,9 @@ struct xtables_globals iptables_xml_globals = { static void print_usage(const char *name, const char *version) __attribute__ ((noreturn)); -static int verbose = 0; +static int verbose; /* Whether to combine actions of sequential rules with identical conditions */ -static int combine = 0; +static int combine; /* Keeping track of external matches and targets. */ static struct option options[] = { {"verbose", 0, NULL, 'v'}, @@ -73,10 +73,10 @@ parse_counters(char *string, struct xt_counters *ctr) /* global new argv and argc */ static char *newargv[255]; -static unsigned int newargc = 0; +static unsigned int newargc; static char *oldargv[255]; -static unsigned int oldargc = 0; +static unsigned int oldargc; /* arg meta data, were they quoted, frinstance */ static int newargvattr[255]; @@ -96,7 +96,7 @@ struct chain { #define maxChains 10240 /* max chains per table */ static struct chain chains[maxChains]; -static int nextChain = 0; +static int nextChain; /* funCtion adding one argument to newargv, updating newargc * returns true if argument added, false otherwise */ diff --git a/iptables/xtables-arp.c b/iptables/xtables-arp.c index 6aa000a..4a968f4 100644 --- a/iptables/xtables-arp.c +++ b/iptables/xtables-arp.c @@ -150,7 +150,7 @@ static struct option original_opts[] = { int RUNTIME_NF_ARP_NUMHOOKS = 3; static struct option *opts = original_opts; -static unsigned int global_option_offset = 0; +static unsigned int global_option_offset; extern void xtables_exit_error(enum xtables_exittype status, const char *msg, ...) __attribute__((noreturn, format(printf,2,3))); struct xtables_globals arptables_globals = { -- 2.9.3 -- To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html