iwlwifi: mei: fix potential NULL-ptr deref
authorJohannes Berg <johannes.berg@intel.com>
Tue, 17 May 2022 09:05:14 +0000 (12:05 +0300)
committerJohannes Berg <johannes.berg@intel.com>
Wed, 18 May 2022 10:58:59 +0000 (12:58 +0200)
If SKB allocation fails, continue rather than using the NULL
pointer.

Coverity CID: 1497650

Cc: stable@vger.kernel.org
Fixes: 2da4366f9e2c ("iwlwifi: mei: add the driver to allow cooperation with CSME")
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Gregory Greenman <gregory.greenman@intel.com>
Link: https://lore.kernel.org/r/20220517120045.90c1b1fd534e.Ibb42463e74d0ec7d36ec81df22e171ae1f6268b0@changeid
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
drivers/net/wireless/intel/iwlwifi/mei/main.c

index 3d2eb15a9662103614e962ed2d0140fc4e728c17..357f14626cf43e66b40fef20d1501f6b2a4a07f2 100644 (file)
@@ -1020,6 +1020,8 @@ static void iwl_mei_handle_sap_data(struct mei_cl_device *cldev,
 
                /* We need enough room for the WiFi header + SNAP + IV */
                skb = netdev_alloc_skb(netdev, len + QOS_HDR_IV_SNAP_LEN);
+               if (!skb)
+                       continue;
 
                skb_reserve(skb, QOS_HDR_IV_SNAP_LEN);
                ethhdr = skb_push(skb, sizeof(*ethhdr));