From mboxrd@z Thu Jan 1 00:00:00 1970 From: Patrick McHardy Subject: Re: [PATCH] netfilter: fix a race in nf_ct_ext_create() Date: Thu, 16 Sep 2010 20:00:04 +0200 Message-ID: <4C925B24.8020900@trash.net> References: <1283269682.2550.111.camel@edumazet-laptop> <20100831155137.GE2421@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: Eric Dumazet , David Miller , netdev , Netfilter Development Mailinglist To: paulmck@linux.vnet.ibm.com Return-path: In-Reply-To: <20100831155137.GE2421@linux.vnet.ibm.com> Sender: netdev-owner@vger.kernel.org List-Id: netfilter-devel.vger.kernel.org On 31.08.2010 17:51, Paul E. McKenney wrote: > On Tue, Aug 31, 2010 at 05:48:02PM +0200, Eric Dumazet wrote: >> > As soon as rcu_read_unlock() is called, there is no guarantee current >> > thread can safely derefence t pointer, rcu protected. >> > >> > Fix is to copy t->alloc_size in a temporary variable. > Yow!!! Good catch!!! > > Reviewed-by: Paul E. McKenney > >> > Signed-off-by: Eric Dumazet Applied, thanks.