]> git.neil.brown.name Git - history.git/commit
[PF_KEY]: spirange should be in host byte order.
authorHerbert Xu <herbert@gondor.apana.org.au>
Sat, 31 Jul 2004 16:39:29 +0000 (09:39 -0700)
committerDavid S. Miller <davem@nuts.davemloft.net>
Sat, 31 Jul 2004 16:39:29 +0000 (09:39 -0700)
commit0797db20ef39f1a80fe14b5f66f4dc36a2683b65
tree87f9b26b87dc0e198a7f6c5592a2448410674c65
parent3860b2810d0f5178a2be62541e516e8d8c968375
[PF_KEY]: spirange should be in host byte order.

I'm looking through the xfrm_alloc_spi stuff and noticed that the
netlink alloc_spi function takes the range in host order while the
PFKEY alloc_spi function takes them in network order.

First I thought that I stuffed up since I was the one who changed
the code in the netlink interface to take them in host order :)

But reading RFC 2367 seems to indicate otherwise.  It says that all
fields are host order unless specified otherwise.  And the spirange
fields are not specified to be network order at all.

Looking at the existing PFKEY users:

User Space
----------
Openswan - Doesn't use PFKEY for this.
Racoon - Puts zeros in there so it doesn't care.  However its test-pfkey
 program stores things in host order.
ISAKMPD - Stores things in host order.

So the conclusion is that we can and should change our PFKEY
implementation to use host order for these fields.

This patch does exactly that.

Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: David S. Miller <davem@redhat.com>
net/key/af_key.c