Skip to contentSkip to conversation

mt76: mt7915: add encap offload for 4-address mode stations

bug report Dan Carpenter 1 participant 1 message Dec 10, 2020 Arm MediaTek

lore
Share & tools
lore
#0 lore raw

Hello Felix Fietkau,

The patch e151d71e503d: "mt76: mt7915: add encap offload for 4-address mode stations" from Aug 21, 2020, leads to the following static checker warning:

	drivers/net/wireless/mediatek/mt76/mt7915/mcu.c:1689 mt7915_mcu_sta_update_hdr_trans()
	error: 'wtbl_hdr' dereferencing possible ERR_PTR()

drivers/net/wireless/mediatek/mt76/mt7915/mcu.c
1676 int mt7915_mcu_sta_update_hdr_trans(struct mt7915_dev *dev,
1677 struct ieee80211_vif *vif,
1678 struct ieee80211_sta *sta)
1679 {

  1680          struct mt7915_sta *msta = (struct mt7915_sta *)sta->drv_priv;
  1681          struct wtbl_req_hdr *wtbl_hdr;
  1682          struct sk_buff *skb;
  1683  
  1684          skb = mt76_mcu_msg_alloc(&dev->mt76, NULL, MT7915_WTBL_UPDATE_MAX_SIZE);
  1685          if (!skb)
  1686                  return -ENOMEM;
  1687  
  1688          wtbl_hdr = mt7915_mcu_alloc_wtbl_req(dev, msta, WTBL_SET, NULL, &skb);
  1689          mt7915_mcu_wtbl_hdr_trans_tlv(skb, vif, sta, NULL, wtbl_hdr);
                                                                   ^^^^^^^^

No check for if mt7915_mcu_alloc_wtbl_req() fails.

1690
1691 return mt76_mcu_skb_send_msg(&dev->mt76, skb, MCU_EXT_CMD_WTBL_UPDATE,
1692 true);
1693 }

regards,
dan carpenter

_______________________________________________
Linux-mediatek mailing list
[email protected]
http://lists.infradead.org/mailman/listinfo/linux-mediatek