Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161736AbbKFTfA (ORCPT ); Fri, 6 Nov 2015 14:35:00 -0500 Received: from mail-wi0-f171.google.com ([209.85.212.171]:36287 "EHLO mail-wi0-f171.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1032012AbbKFTe5 (ORCPT ); Fri, 6 Nov 2015 14:34:57 -0500 From: Dmitry Safonov <0x7f454c46@gmail.com> To: pablo@netfilter.org, kaber@trash.net, kadlec@blackhole.kfki.hu, davem@davemloft.net Cc: netfilter-devel@vger.kernel.org, coreteam@netfilter.org, netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Dmitry Safonov <0x7f454c46@gmail.com> Subject: [PATCH] net: netfilter: fix false positive GCC warnings Date: Fri, 6 Nov 2015 22:35:44 +0300 Message-Id: <1446838544-30487-1-git-send-email-0x7f454c46@gmail.com> X-Mailer: git-send-email 2.6.2 In-Reply-To: <20151106190705.GA20345@breakpoint.cc> References: <20151106190705.GA20345@breakpoint.cc> MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2444 Lines: 62 With x86_64_defconfig: GCC thinks that in nfulnl_recv_config flags parameter is not inited but it was under the same condition (nfula[NFULA_CFG_FLAGS] == true). Suppress this warning: net/netfilter/nfnetlink_log.c: In function ‘nfulnl_recv_config’: net/netfilter/nfnetlink_log.c:320:14: warning: ‘flags’ may be used uninitialized in this function [-Wmaybe-uninitialized] inst->flags = flags; ^ GCC thinks that nfnl_ct wasn't inited, but it is used only under if (ct && diff) where ct is not NULL only when nfnl_ct is inited. Reported-by: Florian Westphal Signed-off-by: Dmitry Safonov <0x7f454c46@gmail.com> --- net/netfilter/nfnetlink_log.c | 2 +- net/netfilter/nfnetlink_queue.c | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/net/netfilter/nfnetlink_log.c b/net/netfilter/nfnetlink_log.c index 06eb48fceb42e4..23f16b99e6388a 100644 --- a/net/netfilter/nfnetlink_log.c +++ b/net/netfilter/nfnetlink_log.c @@ -825,7 +825,7 @@ nfulnl_recv_config(struct sock *ctnl, struct sk_buff *skb, struct net *net = sock_net(ctnl); struct nfnl_log_net *log = nfnl_log_pernet(net); int ret = 0; - u16 flags; + u16 uninitialized_var(flags); if (nfula[NFULA_CFG_CMD]) { u_int8_t pf = nfmsg->nfgen_family; diff --git a/net/netfilter/nfnetlink_queue.c b/net/netfilter/nfnetlink_queue.c index 7d81d280cb4ff3..66b006112921ca 100644 --- a/net/netfilter/nfnetlink_queue.c +++ b/net/netfilter/nfnetlink_queue.c @@ -313,7 +313,7 @@ nfqnl_build_packet_message(struct net *net, struct nfqnl_instance *queue, struct net_device *outdev; struct nf_conn *ct = NULL; enum ip_conntrack_info uninitialized_var(ctinfo); - struct nfnl_ct_hook *nfnl_ct; + struct nfnl_ct_hook *uninitialized_var(nfnl_ct); bool csum_verify; char *secdata = NULL; u32 seclen = 0; @@ -1041,7 +1041,7 @@ nfqnl_recv_verdict(struct sock *ctnl, struct sk_buff *skb, unsigned int verdict; struct nf_queue_entry *entry; enum ip_conntrack_info uninitialized_var(ctinfo); - struct nfnl_ct_hook *nfnl_ct; + struct nfnl_ct_hook *uninitialized_var(nfnl_ct); struct nf_conn *ct = NULL; struct net *net = sock_net(ctnl); -- 2.6.2 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/