Re: [PATCH] openvswitch: Add missing initialization in validate_and_copy_set_tun()

From: Pravin Shelar
Date: Wed Feb 11 2015 - 16:31:30 EST


On Wed, Feb 11, 2015 at 2:23 AM, Geert Uytterhoeven
<geert@xxxxxxxxxxxxxx> wrote:
> net/openvswitch/flow_netlink.c: In function âvalidate_and_copy_set_tunâ:
> net/openvswitch/flow_netlink.c:1749: warning: âerrâ may be used uninitialized in this function
>
> If ipv4_tun_from_nlattr() returns a different positive value than
> OVS_TUNNEL_KEY_ATTR_GENEVE_OPTS, err will be uninitialized, and
> validate_and_copy_set_tun() may return an undefined value instead of a
> zero success indicator. Initialize err to zero to fix this.
>
> Fixes: 1dd144cf5b4b47e1 ("openvswitch: Support VXLAN Group Policy extension")
> Signed-off-by: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>

Acked-by: Pravin B Shelar <pshelar@xxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/