Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content
Commit 1a8f813c authored by Steffen Klassert's avatar Steffen Klassert Committed by Tejaswi Tanikella
Browse files

xfrm: Fix stack-out-of-bounds with misconfigured transport mode policies.



On policies with a transport mode template, we pass the addresses
from the flowi to xfrm_state_find(), assuming that the IP addresses
(and address family) don't change during transformation.

Unfortunately our policy template validation is not strict enough.
It is possible to configure policies with transport mode template
where the address family of the template does not match the selectors
address family. This lead to stack-out-of-bound reads because
we compare arddesses of the wrong family. Fix this by refusing
such a configuration, address family can not change on transport
mode.

We use the assumption that, on transport mode, the first templates
address family must match the address family of the policy selector.
Subsequent transport mode templates must mach the address family of
the previous template.

Change-Id: I33678e32df020045f419f38fc4d955863c42409a
Signed-off-by: default avatarSteffen Klassert <steffen.klassert@secunet.com>
Git-commit: 732706afe1cc46ef48493b3d2b69c98f36314ae4
Git-repo: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git


Signed-off-by: default avatarTejaswi Tanikella <tejaswit@codeaurora.org>
Acked-by: default avatarSharath Chandra Vurukala <sharathv@qti.qualcomm.com>
parent 11f2cc4d
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment