diff options
| author | Andreas Steffen <andreas.steffen@strongswan.org> | 2008-11-04 23:49:19 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-11-04 23:49:19 +0100 |
| commit | 79654a7698195fa043063092f5c1ca5245276fba (patch) | |
| tree | 28089eb142f4d67e241db0bf01990998e922c028 /net/ipv6/xfrm6_state.c | |
| parent | xfrm: Fix xfrm_policy_gc_lock handling. (diff) | |
| download | linux-79654a7698195fa043063092f5c1ca5245276fba.tar.xz linux-79654a7698195fa043063092f5c1ca5245276fba.zip | |
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/xfrm6_state.c')
| -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 89884a4f23aa..60c78cfc2737 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; |