Re: [PATCH v2] net/x25: Fix null-ptr-deref in x25_connect

From: Jakub Kicinski
Date: Fri Nov 06 2020 - 11:06:53 EST


On Fri, 06 Nov 2020 07:23:05 +0100 Martin Schiller wrote:
> On 2020-10-06 07:45, Martin Schiller wrote:
> > This fixes a regression for blocking connects introduced by commit
> > 4becb7ee5b3d ("net/x25: Fix x25_neigh refcnt leak when x25
> > disconnect").
> >
> > The x25->neighbour is already set to "NULL" by x25_disconnect() now,
> > while a blocking connect is waiting in
> > x25_wait_for_connection_establishment(). Therefore x25->neighbour must
> > not be accessed here again and x25->state is also already set to
> > X25_STATE_0 by x25_disconnect().
> >
> > Fixes: 4becb7ee5b3d ("net/x25: Fix x25_neigh refcnt leak when x25
> > disconnect")
> > Signed-off-by: Martin Schiller <ms@xxxxxxxxxx>
>
> @David
> Is there anything left I need to do, to get this fix merged?

Hm, no idea what happened here (you could try to check the state in
patchwork but it's gonna take some digging to find a month old patch).

Please resend and we'll take it from there.