* [PATCH ipsec-next] xfrm: announce deleation of temporary SA
@ 2013-08-23 13:46 Nicolas Dichtel
2013-08-26 10:54 ` Steffen Klassert
0 siblings, 1 reply; 2+ messages in thread
From: Nicolas Dichtel @ 2013-08-23 13:46 UTC (permalink / raw)
To: steffen.klassert, herbert, davem; +Cc: netdev, Nicolas Dichtel
Creation of temporary SA are announced by netlink, but there is no notification
for the deletion.
This patch fix this asymmetric situation.
Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
---
Example:
ip xfrm monitor
acquire proto esp
sel src 10.22.6.51/32 dst 10.24.6.139/32 proto icmp type 8 code 0
policy src 10.22.6.51/32 dst 10.24.6.139/32
dir out priority 3843 ptype main
tmpl src 10.23.6.106 dst 10.23.6.206
proto esp reqid 2147483649 mode tunnel
Expired src 10.23.6.106 dst 10.23.6.206
proto esp spi 0x00000000 reqid 2147483649 mode tunnel
replay-window 0
sel src 10.22.6.51/32 dst 10.24.6.139/32 proto icmp type 8 code 0
hard 1
net/xfrm/xfrm_state.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/xfrm/xfrm_state.c b/net/xfrm/xfrm_state.c
index 4f8ace855864..3fd65b73df7e 100644
--- a/net/xfrm/xfrm_state.c
+++ b/net/xfrm/xfrm_state.c
@@ -471,7 +471,7 @@ expired:
}
err = __xfrm_state_delete(x);
- if (!err && x->id.spi)
+ if (!err)
km_state_expired(x, 1, 0);
xfrm_audit_state_delete(x, err ? 0 : 1,
--
1.8.2.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH ipsec-next] xfrm: announce deleation of temporary SA
2013-08-23 13:46 [PATCH ipsec-next] xfrm: announce deleation of temporary SA Nicolas Dichtel
@ 2013-08-26 10:54 ` Steffen Klassert
0 siblings, 0 replies; 2+ messages in thread
From: Steffen Klassert @ 2013-08-26 10:54 UTC (permalink / raw)
To: Nicolas Dichtel; +Cc: herbert, davem, netdev
On Fri, Aug 23, 2013 at 03:46:08PM +0200, Nicolas Dichtel wrote:
> Creation of temporary SA are announced by netlink, but there is no notification
> for the deletion.
> This patch fix this asymmetric situation.
>
> Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Looks good, applied to ipsec-next. Thanks!
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-08-26 10:54 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-08-23 13:46 [PATCH ipsec-next] xfrm: announce deleation of temporary SA Nicolas Dichtel
2013-08-26 10:54 ` Steffen Klassert
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.