Re: [PATCH] net: fddi: fix UAF in fza_probe
From: Maciej W. Rozycki
Date: Tue Jul 13 2021 - 12:59:58 EST
On Tue, 13 Jul 2021, Pavel Skripkin wrote:
> fp is netdev private data and it cannot be
> used after free_netdev() call. Using fp after free_netdev()
> can cause UAF bug. Fix it by moving free_netdev() after error message.
Can you justify the lines for a better layout? The paragraph looks odd
to me in its current form.
> Fixes: 61414f5ec983 ("FDDI: defza: Add support for DEC FDDIcontroller 700
> TURBOchannel adapter")
> Signed-off-by: Pavel Skripkin <paskripkin@xxxxxxxxx>
Otherwise LGTM. And a good catch, thank you!
Reviewed-by: Maciej W. Rozycki <macro@xxxxxxxxxxx>
Maciej