From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dan Carpenter Subject: Re: [patch v2] ipvs: uninitialized data with IP_VS_IPV6 Date: Tue, 9 Dec 2014 09:49:47 +0300 Message-ID: <20141209064947.GH4912@mwanda> References: <20141206134924.GA17278@mwanda> <20141209005215.GH8897@verge.net.au> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Julian Anastasov , Wensong Zhang , Pablo Neira Ayuso , Patrick McHardy , Jozsef Kadlecsik , "David S. Miller" , netdev@vger.kernel.org, lvs-devel@vger.kernel.org, netfilter-devel@vger.kernel.org, coreteam@netfilter.org, kernel-janitors@vger.kernel.org To: Simon Horman Return-path: Content-Disposition: inline In-Reply-To: <20141209005215.GH8897@verge.net.au> Sender: lvs-devel-owner@vger.kernel.org List-Id: netfilter-devel.vger.kernel.org On Tue, Dec 09, 2014 at 09:52:15AM +0900, Simon Horman wrote: > On Sun, Dec 07, 2014 at 08:39:35PM +0200, Julian Anastasov wrote: > > > > Hello, > > > > On Sat, 6 Dec 2014, Dan Carpenter wrote: > > > > > The app_tcp_pkt_out() function expects "*diff" to be set and ends up > > > using uninitialized data if CONFIG_IP_VS_IPV6 is turned on. > > > > > > The same issue is there in app_tcp_pkt_in(). Thanks to Julian Anastasov > > > for noticing that. > > > > > > Signed-off-by: Dan Carpenter > > > --- > > > v2: fix app_tcp_pkt_in() as well. This is an old bug. > > > > Thanks! It will not break seqs for IPv6 control > > connection, only that we do not support FTP yet :( I have > > the doubt whether this should be classified as bugfix :) > > I guess, it is a net-next material, right? > > Agreed, I have queued it up in ipvs-next. > I'll send a pull request to Pablo if Dan doesn't object > to it going there. No objections from me. regards, dan carpenter