diff options
author | Aditya Pakki <pakki001@umn.edu> | 2019-03-20 13:02:49 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2019-03-21 03:32:57 -0400 |
commit | d70d70aec9632679dd00dcc1b1e8b2517e2c7da0 (patch) | |
tree | f9aabb81d3396895d5558ce6530742f5cb2203a2 | |
parent | 7671ce0d92933762f469266daf43bd34d422d58c (diff) |
staging: rtlwifi: rtl8822b: fix to avoid potential NULL pointer dereference
skb allocated via dev_alloc_skb can fail and return a NULL pointer.
This patch avoids such a scenario and returns, consistent with other
invocations.
Signed-off-by: Aditya Pakki <pakki001@umn.edu>
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/rtlwifi/rtl8822be/fw.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/staging/rtlwifi/rtl8822be/fw.c b/drivers/staging/rtlwifi/rtl8822be/fw.c index f061dd1382aa..cf6b7a80b753 100644 --- a/drivers/staging/rtlwifi/rtl8822be/fw.c +++ b/drivers/staging/rtlwifi/rtl8822be/fw.c | |||
@@ -743,6 +743,8 @@ void rtl8822be_set_fw_rsvdpagepkt(struct ieee80211_hw *hw, bool b_dl_finished) | |||
743 | u1_rsvd_page_loc, 3); | 743 | u1_rsvd_page_loc, 3); |
744 | 744 | ||
745 | skb = dev_alloc_skb(totalpacketlen); | 745 | skb = dev_alloc_skb(totalpacketlen); |
746 | if (!skb) | ||
747 | return; | ||
746 | memcpy((u8 *)skb_put(skb, totalpacketlen), &reserved_page_packet, | 748 | memcpy((u8 *)skb_put(skb, totalpacketlen), &reserved_page_packet, |
747 | totalpacketlen); | 749 | totalpacketlen); |
748 | 750 | ||