[NETFILTER]: Fix NF_QUEUE_NR() parenthesis
authorPatrick McHardy <kaber@trash.net>
Wed, 27 Feb 2008 20:21:18 +0000 (12:21 -0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 27 Feb 2008 20:21:18 +0000 (12:21 -0800)
Properly add parens around the macro argument. This is not needed by
the kernel but the macro is exported to userspace, so it shouldn't
make any assumptions.

Also use NF_VERDICT_BITS instead of NF_VERDICT_QBTIS for the left-shift
since thats whats logically correct.

Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
include/linux/netfilter.h

index b74b615492e8117c47d37f348b13668ef4ffbce5..f0680c2bee73b16cc8b642f1d7de2e1f9133d43c 100644 (file)
@@ -31,7 +31,7 @@
 #define NF_VERDICT_QMASK 0xffff0000
 #define NF_VERDICT_QBITS 16
 
-#define NF_QUEUE_NR(x) (((x << NF_VERDICT_QBITS) & NF_VERDICT_QMASK) | NF_QUEUE)
+#define NF_QUEUE_NR(x) ((((x) << NF_VERDICT_BITS) & NF_VERDICT_QMASK) | NF_QUEUE)
 
 /* only for userspace compatibility */
 #ifndef __KERNEL__