Message ID | 1652957839-127949-1-git-send-email-lyz_cs@pku.edu.cn |
---|---|
State | New |
Headers | show |
Series | mwifiex: Fix potential dereference of NULL pointer | expand |
On 5/19/2022 8:37 AM, Kalle Valo wrote: > Yongzhi Liu <lyz_cs@pku.edu.cn> writes: > Format is wrong, it should be: > > Fixes: 21c5c83ce833 ("mwifiex: support sysfs initiated device coredump") And no blank line between the Fixes tag and the Signed-off-by tag
diff --git a/drivers/net/wireless/marvell/mwifiex/usb.c b/drivers/net/wireless/marvell/mwifiex/usb.c index 8f01fcb..c635206 100644 --- a/drivers/net/wireless/marvell/mwifiex/usb.c +++ b/drivers/net/wireless/marvell/mwifiex/usb.c @@ -686,6 +686,8 @@ static void mwifiex_usb_coredump(struct device *dev) { struct usb_interface *intf = to_usb_interface(dev); struct usb_card_rec *card = usb_get_intfdata(intf); + if (!card->adapter || !card->adapter->priv_num) + return; mwifiex_fw_dump_event(mwifiex_get_priv(card->adapter, MWIFIEX_BSS_ROLE_ANY));
If 'card' is not valid, then we need to check the field 'adapter' and 'priv_num' to avoid use of NULL pointer in function 'mwifiex_get_priv'. Fix this by adding the null pointer check on them. Fixes: 21c5c83ce ("mwifiex: support sysfs initiated device coredump") Signed-off-by: Yongzhi Liu <lyz_cs@pku.edu.cn> --- drivers/net/wireless/marvell/mwifiex/usb.c | 2 ++ 1 file changed, 2 insertions(+)