From mboxrd@z Thu Jan 1 00:00:00 1970 From: Felipe W Damasio Subject: Re: [PATCH] tproxy: nf_tproxy_assign_sock() can handle tw sockets Date: Fri, 9 Jul 2010 15:16:17 -0300 Message-ID: References: <1278626921.2435.73.camel@edumazet-laptop> <1278695580.2696.55.camel@edumazet-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Cc: David Miller , Patrick McHardy , linux-kernel@vger.kernel.org, netdev To: Eric Dumazet Return-path: In-Reply-To: <1278695580.2696.55.camel@edumazet-laptop> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org Hi, 2010/7/9 Eric Dumazet : > Reviewing tproxy stuff I spotted a problem in nf_tproxy_assign_sock() > but I could not see how it could explain your crash. > > We can read uninitialized memory and trigger a fault in > nf_tproxy_assign_sock(), not later in tcp_recvmsg()... Well, since I can't reproduce the bug, if you think this patch solves my problem, I'll tell my bosses that we can put back the production machine online. But anyway, if there's test you think I can run, I have same the same hardware on a lab using the same setup as the production environment... Thanks, Felipe Damasio