mirror of
https://github.com/hardkernel/linux.git
synced 2026-06-08 03:40:35 +09:00
net: mctp i2c: Copy headers if cloned
[ Upstream commit df8ce77ba8b7c012a3edd1ca7368b46831341466 ]
Use skb_cow_head() prior to modifying the TX SKB. This is necessary
when the SKB has been cloned, to avoid modifying other shared clones.
Signed-off-by: Matt Johnston <matt@codeconstruct.com.au>
Fixes: f5b8abf9fc ("mctp i2c: MCTP I2C binding driver")
Link: https://patch.msgid.link/20250306-matt-mctp-i2c-cow-v1-1-293827212681@codeconstruct.com.au
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
95b5304073
commit
c49e91520d
@@ -543,6 +543,7 @@ static int mctp_i2c_header_create(struct sk_buff *skb, struct net_device *dev,
|
|||||||
struct mctp_i2c_hdr *hdr;
|
struct mctp_i2c_hdr *hdr;
|
||||||
struct mctp_hdr *mhdr;
|
struct mctp_hdr *mhdr;
|
||||||
u8 lldst, llsrc;
|
u8 lldst, llsrc;
|
||||||
|
int rc;
|
||||||
|
|
||||||
if (len > MCTP_I2C_MAXMTU)
|
if (len > MCTP_I2C_MAXMTU)
|
||||||
return -EMSGSIZE;
|
return -EMSGSIZE;
|
||||||
@@ -553,6 +554,10 @@ static int mctp_i2c_header_create(struct sk_buff *skb, struct net_device *dev,
|
|||||||
lldst = *((u8 *)daddr);
|
lldst = *((u8 *)daddr);
|
||||||
llsrc = *((u8 *)saddr);
|
llsrc = *((u8 *)saddr);
|
||||||
|
|
||||||
|
rc = skb_cow_head(skb, sizeof(struct mctp_i2c_hdr));
|
||||||
|
if (rc)
|
||||||
|
return rc;
|
||||||
|
|
||||||
skb_push(skb, sizeof(struct mctp_i2c_hdr));
|
skb_push(skb, sizeof(struct mctp_i2c_hdr));
|
||||||
skb_reset_mac_header(skb);
|
skb_reset_mac_header(skb);
|
||||||
hdr = (void *)skb_mac_header(skb);
|
hdr = (void *)skb_mac_header(skb);
|
||||||
|
|||||||
Reference in New Issue
Block a user