From mboxrd@z Thu Jan 1 00:00:00 1970 From: Pablo Neira Ayuso Subject: Re: [PATCH] netfilter: SYNPROXY core: fix warning in __nf_ct_ext_add_length() Date: Wed, 4 Sep 2013 14:56:39 +0200 Message-ID: <20130904125639.GB4520@localhost> References: <1377765129-8490-1-git-send-email-kaber@trash.net> <20130829123852.7b1c0093@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Patrick McHardy , netfilter-devel@vger.kernel.org, mph@one.com To: Jesper Dangaard Brouer Return-path: Received: from mail.us.es ([193.147.175.20]:35406 "EHLO mail.us.es" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1757098Ab3IDM4r (ORCPT ); Wed, 4 Sep 2013 08:56:47 -0400 Content-Disposition: inline In-Reply-To: <20130829123852.7b1c0093@redhat.com> Sender: netfilter-devel-owner@vger.kernel.org List-ID: On Thu, Aug 29, 2013 at 12:38:52PM +0200, Jesper Dangaard Brouer wrote: > On Thu, 29 Aug 2013 10:32:09 +0200 > Patrick McHardy wrote: > > > With CONFIG_NETFILTER_DEBUG we get the following warning during SYNPROXY init: > > > > [ 80.558906] WARNING: CPU: 1 PID: 4833 at net/netfilter/nf_conntrack_extend.c:80 __nf_ct_ext_add_length+0x217/0x220 [nf_conntrack]() > > > > The reason is that the conntrack template is set to confirmed before adding > > the extension and it is invalid to add extensions to already confirmed > > conntracks. Fix by adding the extensions before setting the conntrack to > > confirmed. > > > > Reported-by: Jesper Dangaard Brouer > > Signed-off-by: Patrick McHardy > > Acked-by: Jesper Dangaard Brouer Applied, thanks.