Thread (8 messages) 8 messages, 2 authors, 2022-09-30

Re: [PATCH v2 ipsec 2/2] xfrm: Ensure policy checked for nested ESP tunnels

From: Steffen Klassert <steffen.klassert@secunet.com>
Date: 2022-09-22 06:27:28

On Fri, Sep 16, 2022 at 10:44:42PM -0700, Benedict Wong wrote:
Thanks for the response; apologies for taking a while to re-patch this
and verify.

I think this /almost/ does what we need to. I'm still seeing v6 ESP in v6
ESP tunnels failing; I think it's due to the fact that the IPv6 ESP
codepath does not trigger policy checks in the receive codepath until it
hits the socket, or changes namespace.
Perhaps if we verify policy unconditionally in xfrmi_rcv_cb? combined
with your change above, this should ensure IPv6 ESP also checks policies,
and inside that clear the secpath?
Hm, do you know why this is different to IPv4? IPv4 and IPv6 should
do the same regarding to policy checks.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help