From mboxrd@z Thu Jan 1 00:00:00 1970 From: "gregkh@linuxfoundation.org" Subject: Re: [PATCH] net: fix neigh_resolve_output can cause skb_under_panic Date: Fri, 28 Sep 2012 16:52:09 -0700 Message-ID: <20120928235209.GA11474@kroah.com> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "davem@davemloft.net" , "michel@digirati.com.br" , "eric.dumazet@gmail.com" To: Ramesh Nagappa Return-path: Received: from out1-smtp.messagingengine.com ([66.111.4.25]:32885 "EHLO out1-smtp.messagingengine.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S964876Ab2I1XwL (ORCPT ); Fri, 28 Sep 2012 19:52:11 -0400 Content-Disposition: inline In-Reply-To: Sender: netdev-owner@vger.kernel.org List-ID: On Fri, Sep 28, 2012 at 07:40:39PM -0400, Ramesh Nagappa wrote: > >From fd023edd911ef12aca38a72b40241661c202684f Mon Sep 17 00:00:00 2001 > From: Ramesh Nagappa > Date: Thu, 27 Sep 2012 10:20:58 -0700 > Subject: [PATCH] net: fix neigh_resolve_output can cause skb_under_panic Why is all of this in the middle of the changelog section? I'm guessing you didn't use 'git send-email' for this? And why are you copying me on the patch? > The retry loop in the neigh_resolve_output() and neigh_connected_output() can add > a hard_header without resetting the skb to network header. This causes the > skb_push() in dev_hard_header() to fail. > Signed-off-by: Ramesh Nagappa > Signed-off-by: Shawn Lu > Reviewed-by: Billie Alsup > Reviewed-by: Robert Coulson You need a blank line before the first Signed-off-by: line. Surely one of the reviewers should have caught this basic thing? greg k-h