batman-adv: fix OOB read/write in network-coding decode
authorStanislav Fort <stanislav.fort@aisle.com>
Sun, 31 Aug 2025 14:56:23 +0000 (16:56 +0200)
committerSimon Wunderlich <sw@simonwunderlich.de>
Sun, 31 Aug 2025 15:01:35 +0000 (17:01 +0200)
commitd77b6ff0ce35a6d0b0b7b9581bc3f76d041d4087
tree4870a24b7ab279ae6b356dc2f67a46052f9c638e
parent6439a0e64c355d2e375bd094f365d56ce81faba3
batman-adv: fix OOB read/write in network-coding decode

batadv_nc_skb_decode_packet() trusts coded_len and checks only against
skb->len. XOR starts at sizeof(struct batadv_unicast_packet), reducing
payload headroom, and the source skb length is not verified, allowing an
out-of-bounds read and a small out-of-bounds write.

Validate that coded_len fits within the payload area of both destination
and source sk_buffs before XORing.

Fixes: 2df5278b0267 ("batman-adv: network coding - receive coded packets and decode them")
Cc: stable@vger.kernel.org
Reported-by: Stanislav Fort <disclosure@aisle.com>
Signed-off-by: Stanislav Fort <stanislav.fort@aisle.com>
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de>
net/batman-adv/network-coding.c