]> git.baikalelectronics.ru Git - kernel.git/commit
[IPSEC]: Avoid undefined shift operation when testing algorithm ID
authorHerbert Xu <herbert@gondor.apana.org.au>
Thu, 20 Dec 2007 07:44:29 +0000 (23:44 -0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 20 Dec 2007 07:44:29 +0000 (23:44 -0800)
commitd1be4f1a8f88afc5b1c2ea5e83a5d7fb214d695e
tree861e4cffa93b61d1469df346267fa068f9fdf283
parented81da8c187a2709a931bc5c7e464c67da5cc834
[IPSEC]: Avoid undefined shift operation when testing algorithm ID

The aalgos/ealgos fields are only 32 bits wide.  However, af_key tries
to test them with the expression 1 << id where id can be as large as
253.  This produces different behaviour on different architectures.

The following patch explicitly checks whether ID is greater than 31
and fails the check if that's the case.

We cannot easily extend the mask to be longer than 32 bits due to
exposure to user-space.  Besides, this whole interface is obsolete
anyway in favour of the xfrm_user interface which doesn't use this
bit mask in templates (well not within the kernel anyway).

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