diff options
| author | Herbert Xu <herbert@gondor.apana.org.au> | 2007-07-31 11:04:32 +0200 |
|---|---|---|
| committer | David S. Miller <davem@sunset.davemloft.net> | 2007-07-31 11:28:32 +0200 |
| commit | 196b003620f1ee8d0fc63f13f341187d63c1dc0a (patch) | |
| tree | 2710ea74dab44caa9d3405fc6d95a2663f253f16 /net/xfrm | |
| parent | [TCP]: Bidir flow must not disregard SACK blocks for lost marking (diff) | |
| download | linux-196b003620f1ee8d0fc63f13f341187d63c1dc0a.tar.xz linux-196b003620f1ee8d0fc63f13f341187d63c1dc0a.zip | |
[IPSEC]: Ensure that state inner family is set
Similar to the issue we had with template families which
specified the inner families of policies, we need to set
the inner families of states as the main xfrm user Openswan
leaves it as zero.
af_key is unaffected because the inner family is set by it
and not the KM.
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/xfrm')
| -rw-r--r-- | net/xfrm/xfrm_user.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/net/xfrm/xfrm_user.c b/net/xfrm/xfrm_user.c index c06883bf620e..61339e17a0f5 100644 --- a/net/xfrm/xfrm_user.c +++ b/net/xfrm/xfrm_user.c @@ -322,6 +322,13 @@ static void copy_from_user_state(struct xfrm_state *x, struct xfrm_usersa_info * x->props.family = p->family; memcpy(&x->props.saddr, &p->saddr, sizeof(x->props.saddr)); x->props.flags = p->flags; + + /* + * Set inner address family if the KM left it as zero. + * See comment in validate_tmpl. + */ + if (!x->sel.family) + x->sel.family = p->family; } /* |