ipv6: check return value of ipv6_skip_exthdr
authorJordy Zomer <jordy@pwning.systems>
Wed, 17 Nov 2021 19:06:48 +0000 (20:06 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 8 Dec 2021 08:03:19 +0000 (09:03 +0100)
commit4d5968ea06fbfc7f6bb88dee7ce459f4755cebb5
tree8167ac0b2a0f99f203f3e1048340ef8cb11ee117
parent22519eff7df2d88adcc2568d86046ce1e2b52803
ipv6: check return value of ipv6_skip_exthdr

[ Upstream commit 5f9c55c8066bcd93ac25234a02585701fe2e31df ]

The offset value is used in pointer math on skb->data.
Since ipv6_skip_exthdr may return -1 the pointer to uh and th
may not point to the actual udp and tcp headers and potentially
overwrite other stuff. This is why I think this should be checked.

EDIT:  added {}'s, thanks Kees

Signed-off-by: Jordy Zomer <jordy@pwning.systems>
Signed-off-by: David S. Miller <davem@davemloft.net>
Signed-off-by: Sasha Levin <sashal@kernel.org>
net/ipv6/esp6.c