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
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.