Re: [PATCH] qtnfmac: fix NULL pointer dereference

From: Kalle Valo
Date: Fri Jun 01 2018 - 11:53:03 EST


"Gustavo A. R. Silva" <gustavo@xxxxxxxxxxxxxx> writes:

> In case *vif* is NULL at 655: if (!vif), the execution path jumps to
> label out, where *vif* is dereferenced at 679:
>
> if (vif->sta_state == QTNF_STA_CONNECTING)
>
> Fix this by immediately returning when *vif* is NULL instead of
> jumping to label out.
>
> Addresses-Coverity-ID: 1469567 ("Dereference after null check")
> Fixes: 480daa9cb62c ("qtnfmac: fix invalid STA state on EAPOL failure")
> Signed-off-by: Gustavo A. R. Silva <gustavo@xxxxxxxxxxxxxx>

As commit 480daa9cb62c was recently applied to wireless-drivers-next
I'll queue this to 4.18.

--
Kalle Valo