diff options
author | Andreas Steffen <andreas.steffen@strongswan.org> | 2008-11-04 14:49:19 -0800 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-11-04 14:49:19 -0800 |
commit | 79654a7698195fa043063092f5c1ca5245276fba (patch) | |
tree | 28089eb142f4d67e241db0bf01990998e922c028 /net/ipv6 | |
parent | bbb770e7ab9a436752babfc8765e422d7481be1f (diff) | |
download | op-kernel-dev-79654a7698195fa043063092f5c1ca5245276fba.zip op-kernel-dev-79654a7698195fa043063092f5c1ca5245276fba.tar.gz |
xfrm: Have af-specific init_tempsel() initialize family field of temporary selector
While adding MIGRATE support to strongSwan, Andreas Steffen noticed that
the selectors provided in XFRM_MSG_ACQUIRE have their family field
uninitialized (those in MIGRATE do have their family set).
Looking at the code, this is because the af-specific init_tempsel()
(called via afinfo->init_tempsel() in xfrm_init_tempsel()) do not set
the value.
Reported-by: Andreas Steffen <andreas.steffen@strongswan.org>
Acked-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Arnaud Ebalard <arno@natisbad.org>
Diffstat (limited to 'net/ipv6')
-rw-r--r-- | net/ipv6/xfrm6_state.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/ipv6/xfrm6_state.c b/net/ipv6/xfrm6_state.c index 89884a4..60c78cf 100644 --- a/net/ipv6/xfrm6_state.c +++ b/net/ipv6/xfrm6_state.c @@ -34,6 +34,7 @@ __xfrm6_init_tempsel(struct xfrm_state *x, struct flowi *fl, x->sel.dport_mask = htons(0xffff); x->sel.sport = xfrm_flowi_sport(fl); x->sel.sport_mask = htons(0xffff); + x->sel.family = AF_INET6; x->sel.prefixlen_d = 128; x->sel.prefixlen_s = 128; x->sel.proto = fl->proto; |