> -----Original Message----- > From: Colin King [mailto:colin.king@xxxxxxxxxxxxx] > Sent: Saturday, March 25, 2017 10:17 AM > To: KY Srinivasan <kys@xxxxxxxxxxxxx>; Haiyang Zhang > <haiyangz@xxxxxxxxxxxxx>; Stephen Hemminger <sthemmin@xxxxxxxxxxxxx>; > devel@xxxxxxxxxxxxxxxxxxxxxx; netdev@xxxxxxxxxxxxxxx > Cc: kernel-janitors@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx > Subject: [PATCH] netvsc: fix unititialized return value in variable ret > > From: Colin Ian King <colin.king@xxxxxxxxxxxxx> > > 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.king@xxxxxxxxxxxxx> > --- > 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. _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
