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