summaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorPatrick McHardy <kaber@trash.net>2008-02-27 12:21:18 -0800
committerDavid S. Miller <davem@davemloft.net>2008-02-27 12:21:18 -0800
commitfbabbed8284d1526ed01754ecd4fabdb941a1ff2 (patch)
treeddfe8bc9753436b0d1ecae3c464bb382088e9b86 /include
parent6556874dc3770aefae89907b3cf9be8e23d66137 (diff)
downloadop-kernel-dev-fbabbed8284d1526ed01754ecd4fabdb941a1ff2.zip
op-kernel-dev-fbabbed8284d1526ed01754ecd4fabdb941a1ff2.tar.gz
[NETFILTER]: Fix NF_QUEUE_NR() parenthesis
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>
Diffstat (limited to 'include')
-rw-r--r--include/linux/netfilter.h2
1 files changed, 1 insertions, 1 deletions
diff --git a/include/linux/netfilter.h b/include/linux/netfilter.h
index b74b615..f0680c2 100644
--- a/include/linux/netfilter.h
+++ b/include/linux/netfilter.h
@@ -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__
OpenPOWER on IntegriCloud