mirror of
https://github.com/nxp-imx/linux-imx.git
synced 2025-07-14 05:09:36 +02:00
wifi: mt76: replace skb_put with skb_put_zero
[ Upstream commit 7f819a2f4f
]
Avoid potentially reusing uninitialized data
Signed-off-by: Felix Fietkau <nbd@nbd.name>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
9a24eb8010
commit
ff6b26be13
|
@ -255,7 +255,7 @@ mt76_connac_mcu_add_nested_tlv(struct sk_buff *skb, int tag, int len,
|
||||||
};
|
};
|
||||||
u16 ntlv;
|
u16 ntlv;
|
||||||
|
|
||||||
ptlv = skb_put(skb, len);
|
ptlv = skb_put_zero(skb, len);
|
||||||
memcpy(ptlv, &tlv, sizeof(tlv));
|
memcpy(ptlv, &tlv, sizeof(tlv));
|
||||||
|
|
||||||
ntlv = le16_to_cpu(ntlv_hdr->tlv_num);
|
ntlv = le16_to_cpu(ntlv_hdr->tlv_num);
|
||||||
|
@ -1654,7 +1654,7 @@ int mt76_connac_mcu_hw_scan(struct mt76_phy *phy, struct ieee80211_vif *vif,
|
||||||
set_bit(MT76_HW_SCANNING, &phy->state);
|
set_bit(MT76_HW_SCANNING, &phy->state);
|
||||||
mvif->scan_seq_num = (mvif->scan_seq_num + 1) & 0x7f;
|
mvif->scan_seq_num = (mvif->scan_seq_num + 1) & 0x7f;
|
||||||
|
|
||||||
req = (struct mt76_connac_hw_scan_req *)skb_put(skb, sizeof(*req));
|
req = (struct mt76_connac_hw_scan_req *)skb_put_zero(skb, sizeof(*req));
|
||||||
|
|
||||||
req->seq_num = mvif->scan_seq_num | mvif->band_idx << 7;
|
req->seq_num = mvif->scan_seq_num | mvif->band_idx << 7;
|
||||||
req->bss_idx = mvif->idx;
|
req->bss_idx = mvif->idx;
|
||||||
|
@ -1782,7 +1782,7 @@ int mt76_connac_mcu_sched_scan_req(struct mt76_phy *phy,
|
||||||
|
|
||||||
mvif->scan_seq_num = (mvif->scan_seq_num + 1) & 0x7f;
|
mvif->scan_seq_num = (mvif->scan_seq_num + 1) & 0x7f;
|
||||||
|
|
||||||
req = (struct mt76_connac_sched_scan_req *)skb_put(skb, sizeof(*req));
|
req = (struct mt76_connac_sched_scan_req *)skb_put_zero(skb, sizeof(*req));
|
||||||
req->version = 1;
|
req->version = 1;
|
||||||
req->seq_num = mvif->scan_seq_num | mvif->band_idx << 7;
|
req->seq_num = mvif->scan_seq_num | mvif->band_idx << 7;
|
||||||
|
|
||||||
|
@ -2416,7 +2416,7 @@ int mt76_connac_mcu_update_gtk_rekey(struct ieee80211_hw *hw,
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
skb_put_data(skb, &hdr, sizeof(hdr));
|
skb_put_data(skb, &hdr, sizeof(hdr));
|
||||||
gtk_tlv = (struct mt76_connac_gtk_rekey_tlv *)skb_put(skb,
|
gtk_tlv = (struct mt76_connac_gtk_rekey_tlv *)skb_put_zero(skb,
|
||||||
sizeof(*gtk_tlv));
|
sizeof(*gtk_tlv));
|
||||||
gtk_tlv->tag = cpu_to_le16(UNI_OFFLOAD_OFFLOAD_GTK_REKEY);
|
gtk_tlv->tag = cpu_to_le16(UNI_OFFLOAD_OFFLOAD_GTK_REKEY);
|
||||||
gtk_tlv->len = cpu_to_le16(sizeof(*gtk_tlv));
|
gtk_tlv->len = cpu_to_le16(sizeof(*gtk_tlv));
|
||||||
|
@ -2539,7 +2539,7 @@ mt76_connac_mcu_set_wow_pattern(struct mt76_dev *dev,
|
||||||
return -ENOMEM;
|
return -ENOMEM;
|
||||||
|
|
||||||
skb_put_data(skb, &hdr, sizeof(hdr));
|
skb_put_data(skb, &hdr, sizeof(hdr));
|
||||||
ptlv = (struct mt76_connac_wow_pattern_tlv *)skb_put(skb, sizeof(*ptlv));
|
ptlv = (struct mt76_connac_wow_pattern_tlv *)skb_put_zero(skb, sizeof(*ptlv));
|
||||||
ptlv->tag = cpu_to_le16(UNI_SUSPEND_WOW_PATTERN);
|
ptlv->tag = cpu_to_le16(UNI_SUSPEND_WOW_PATTERN);
|
||||||
ptlv->len = cpu_to_le16(sizeof(*ptlv));
|
ptlv->len = cpu_to_le16(sizeof(*ptlv));
|
||||||
ptlv->data_len = pattern->pattern_len;
|
ptlv->data_len = pattern->pattern_len;
|
||||||
|
|
|
@ -422,7 +422,7 @@ mt7915_mcu_add_nested_subtlv(struct sk_buff *skb, int sub_tag, int sub_len,
|
||||||
.len = cpu_to_le16(sub_len),
|
.len = cpu_to_le16(sub_len),
|
||||||
};
|
};
|
||||||
|
|
||||||
ptlv = skb_put(skb, sub_len);
|
ptlv = skb_put_zero(skb, sub_len);
|
||||||
memcpy(ptlv, &tlv, sizeof(tlv));
|
memcpy(ptlv, &tlv, sizeof(tlv));
|
||||||
|
|
||||||
le16_add_cpu(sub_ntlv, 1);
|
le16_add_cpu(sub_ntlv, 1);
|
||||||
|
|
Loading…
Reference in New Issue
Block a user