> -----Original Message----- > From: Colin King [mailto:colin.k...@canonical.com] > Sent: Saturday, March 25, 2017 10:17 AM > To: KY Srinivasan <k...@microsoft.com>; Haiyang Zhang > <haiya...@microsoft.com>; Stephen Hemminger <sthem...@microsoft.com>; > de...@linuxdriverproject.org; netdev@vger.kernel.org > Cc: kernel-janit...@vger.kernel.org; linux-ker...@vger.kernel.org > Subject: [PATCH] netvsc: fix unititialized return value in variable ret > > From: Colin Ian King <colin.k...@canonical.com> > > It is possible for an uninitialized value of ret to be returned > so fix this by initializing ret to zero. > > Detected by CoverityScan, CID#1420762 ("Uninitialized scalar variable") > > Fixes: 163891d7d429 ("netvsc: handle offline mtu and channel change") > Signed-off-by: Colin Ian King <colin.k...@canonical.com> > --- > drivers/net/hyperv/netvsc_drv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/net/hyperv/netvsc_drv.c > b/drivers/net/hyperv/netvsc_drv.c > index eb7ae79d47bb..f830bbbd8ad4 100644 > --- a/drivers/net/hyperv/netvsc_drv.c > +++ b/drivers/net/hyperv/netvsc_drv.c > @@ -855,7 +855,7 @@ static int netvsc_change_mtu(struct net_device *ndev, > int mtu) > struct hv_device *hdev = ndevctx->device_ctx; > struct netvsc_device_info device_info; > bool was_running; > - int ret; > + int ret = 0;
This was just fix by commit: 386f57622cc3bbb39b9c00fc8762bc9ab28e0f8d Thanks.
