brcmfmac: uninitialized "ret" variable

Dan Carpenter · Kalle Valo · commit 977bc523000d · 2016-04-06T21:27:16.000+03:00
There is an error path where "ret" isn't initialized.

Signed-off-by: Dan Carpenter &lt;dan.carpenter@oracle.com&gt;
Signed-off-by: Kalle Valo &lt;kvalo@codeaurora.org&gt;
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c
@@ -250,7 +250,7 @@ static int brcmf_sdiod_request_data(struct brcmf_sdio_dev *sdiodev, u8 fn,
 				    u32 addr, u8 regsz, void *data, bool write)
 {
 	struct sdio_func *func;
-	int ret;
+	int ret = -EINVAL;
 
 	brcmf_dbg(SDIO, "rw=%d, func=%d, addr=0x%05x, nbytes=%d\n",
 		  write, fn, addr, regsz);

Original file line number	Diff line number	Diff line change
`@@ -250,7 +250,7 @@ static int brcmf_sdiod_request_data(struct brcmf_sdio_dev *sdiodev, u8 fn,`
`250`	`250`	`u32 addr, u8 regsz, void *data, bool write)`
`251`	`251`	`{`
`252`	`252`	`struct sdio_func *func;`
`253`		`- int ret;`
	`253`	`+ int ret = -EINVAL;`
`254`	`254`
`255`	`255`	`brcmf_dbg(SDIO, "rw=%d, func=%d, addr=0x%05x, nbytes=%d\n",`
`256`	`256`	`write, fn, addr, regsz);`