Re: [PATCH] net: mscc: ocelot: Fix uninitialized error in ocelot_netdevice_event()

From: David Miller
Date: Thu Jun 07 2018 - 16:55:26 EST


From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
Date: Thu, 7 Jun 2018 15:10:30 +0200

> With gcc-4.1.2:
>
> drivers/net/ethernet/mscc/ocelot.c: In function ʽocelot_netdevice_eventʼ:
> drivers/net/ethernet/mscc/ocelot.c:1129: warning: ʽretʼ may be used uninitialized in this function
>
> If the list iterated over by netdev_for_each_lower_dev() is empty, ret
> is never initialized, and converted into a notifier return value.
>
> Fix this by preinitializing ret to zero.
>
> Fixes: a556c76adc052c97 ("net: mscc: Add initial Ocelot switch support")
> Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> ---
> This may be unlikely to happen, but given the notifier is called for any
> (also non-ocelot) network device, better be safe than sorry.

Applied.