On Tue, 2012-11-06 at 13:28 +0100, Greg KH wrote: > On Tue, Nov 06, 2012 at 01:10:10PM +0100, Eric Leblond wrote: > > Due to a NULL dereference, the following patch is causing oops > > in normal trafic condition: > > > > commit c0de08d04215031d68fa13af36f347a6cfa252ca > > Author: Eric Leblond <[email protected]> > > Date: Thu Aug 16 22:02:58 2012 +0000 > > > > af_packet: don't emit packet on orig fanout group > > > > This buggy patch was a feature fix and has reached most stable > > branches. > > > > When skb->sk is NULL and when packet fanout is used, there is a > > crash in match_fanout_group where skb->sk is accessed. > > This patch fixes the issue by returning false as soon as the > > socket is NULL: this correspond to the wanted behavior because > > the kernel as to resend the skb to all the listening socket in > > this case. > > > > Signed-off-by: Eric Leblond <[email protected]> > > --- > > net/core/dev.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > <formletter> > > This is not the correct way to submit patches for inclusion in the > stable kernel tree. Please read Documentation/stable_kernel_rules.txt > for how to do this properly. > > </formletter>
Right. By the way, David Miller prefers to handle stable submissions himself. So Eric, I guess you could just do nothing for this particular patch, since your [email protected] addition in the CC will be void. (Dont add the Cc: in the patch itself, as David will remove it anyway) Thanks -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
