diff options
| author | Andy Duan <fugang.duan@nxp.com> | 2018-11-07 15:06:03 +0800 |
|---|---|---|
| committer | Jason Liu <jason.hui.liu@nxp.com> | 2019-02-12 10:35:03 +0800 |
| commit | fc490900557f91b647a387eeaf33047b7a313daf (patch) | |
| tree | 3551e5707ee834a5336b6cefd5395da1142030d3 /drivers/net/wireless/broadcom/brcm80211/brcmfmac | |
| parent | edf54ecf6db4021108509b18b82a3eceb0278ded (diff) | |
MLK-20190-03 brcmfmac: check thre return value of .brcmf_fil_iovar_data_get()
Check the return value of .brcmf_fil_iovar_data_get() to fix
the coverity issue of "error handling issue".
Reviewed-by: Haibo Chen <haibo.chen@nxp.com>
Signed-off-by: Fugang Duan <fugang.duan@nxp.com>
Diffstat (limited to 'drivers/net/wireless/broadcom/brcm80211/brcmfmac')
| -rw-r--r-- | drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c index 169714d3e677..62f1eedc65c2 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/sdio.c @@ -2560,12 +2560,15 @@ brcmf_sdio_ulp_preinit(struct device *dev) struct brcmf_bus *bus_if = dev_get_drvdata(dev); struct brcmf_sdio_dev *sdiodev = bus_if->bus_priv.sdio; struct brcmf_if *ifp = bus_if->drvr->iflist[0]; + s32 err = 0; brcmf_dbg(TRACE, "Enter\n"); /* Query ulp_sdioctrl iovar to get the ULP related SHM offsets */ - brcmf_fil_iovar_data_get(ifp, "ulp_sdioctrl", &sdiodev->shm_ulp, - sizeof(sdiodev->shm_ulp)); + err = brcmf_fil_iovar_data_get(ifp, "ulp_sdioctrl", &sdiodev->shm_ulp, + sizeof(sdiodev->shm_ulp)); + if (err) + brcmf_err("ulp_sdioctrl iovar returned err = %d\n", err); sdiodev->ulp = false; |