diff options
author | Zhaoyang Liu <liuzy@marvell.com> | 2015-06-05 01:01:53 +0530 |
---|---|---|
committer | Kalle Valo <kvalo@codeaurora.org> | 2015-06-08 11:49:45 +0300 |
commit | 7a56c4168e4453732c35b1e33a683d91f04f379f (patch) | |
tree | 87646969f3dcef192cdbc00810573b8edba93cd6 /drivers/net/wireless | |
parent | ca7bdd95b26d1424e925f17eb546a491d93afd49 (diff) | |
download | op-kernel-dev-7a56c4168e4453732c35b1e33a683d91f04f379f.zip op-kernel-dev-7a56c4168e4453732c35b1e33a683d91f04f379f.tar.gz |
mwifiex: change debug dump issue since skb maybe null
This patch fixes semantic warning for debugging data dump feature.
Previous code is based on the assumption that skb is not null.
New change makes sure that we already have data buffer.
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Zhaoyang Liu <liuzy@marvell.com>
Signed-off-by: Avinash Patil <patila@marvell.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
Diffstat (limited to 'drivers/net/wireless')
-rw-r--r-- | drivers/net/wireless/mwifiex/cmdevt.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/net/wireless/mwifiex/cmdevt.c b/drivers/net/wireless/mwifiex/cmdevt.c index a51feac..207da40 100644 --- a/drivers/net/wireless/mwifiex/cmdevt.c +++ b/drivers/net/wireless/mwifiex/cmdevt.c @@ -469,10 +469,11 @@ int mwifiex_process_event(struct mwifiex_adapter *adapter) memset(rx_info, 0, sizeof(*rx_info)); rx_info->bss_num = priv->bss_num; rx_info->bss_type = priv->bss_type; + mwifiex_dbg_dump(adapter, EVT_D, "Event Buf:", + skb->data, skb->len); } mwifiex_dbg(adapter, EVENT, "EVENT: cause: %#x\n", eventcause); - mwifiex_dbg_dump(adapter, EVT_D, "Event Buf:", skb->data, skb->len); if (priv->bss_role == MWIFIEX_BSS_ROLE_UAP) ret = mwifiex_process_uap_event(priv); |