Re: kernel BUG at net/key/af_key.c:LINE!

From: Eric Biggers
Date: Sun Dec 03 2017 - 15:28:31 EST


On Wed, Nov 15, 2017 at 12:29:19PM +0100, Steffen Klassert wrote:
> On Fri, Nov 10, 2017 at 02:14:06PM +1100, Herbert Xu wrote:
> > On Fri, Nov 10, 2017 at 01:30:38PM +1100, Herbert Xu wrote:
> > >
> > > I found the problem. This crap is coming from clone_policy. Now
> > > let me where this code came from.
> >
> > ---8<---
> > Subject: xfrm: Copy policy family in clone_policy
> >
> > The syzbot found an ancient bug in the IPsec code. When we cloned
> > a socket policy (for example, for a child TCP socket derived from a
> > listening socket), we did not copy the family field. This results
> > in a live policy with a zero family field. This triggers a BUG_ON
> > check in the af_key code when the cloned policy is retrieved.
> >
> > This patch fixes it by copying the family field over.
> >
> > Reported-by: syzbot <syzkaller@xxxxxxxxxxxxxxxx>
> > Signed-off-by: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx>
>
> Patch applied, thanks Herbert!

And to tell the bot what fixes this:

#syz fix: xfrm: Copy policy family in clone_policy

Also, does this fix need to go to stable? The commit doesn't have Cc: stable.