summaryrefslogtreecommitdiffstats
path: root/kernel
diff options
context:
space:
mode:
authorPatrick McHardy <kaber@trash.net>2011-02-02 20:40:12 +0100
committerJozsef Kadlecsik <kadlec@blackhole.kfki.hu>2011-02-02 20:40:12 +0100
commit85cee26417bb1456575028fdfe343b3db777a5c9 (patch)
treec22ba5a0d36fd7828e8e0fa848a449191b8712c5 /kernel
parentad80ed358b507d5159e1212673e7c7811e9fa539 (diff)
netfilter: ipset: add missing break statemtns in ip_set_get_ip_port()
Don't fall through in the switch statement, otherwise IPv4 headers are incorrectly parsed again as IPv6 and the return value will always be 'false'. Signed-off-by: Patrick McHardy <kaber@trash.net>
Diffstat (limited to 'kernel')
-rw-r--r--kernel/ip_set_getport.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/kernel/ip_set_getport.c b/kernel/ip_set_getport.c
index 76737bb..4dd2785 100644
--- a/kernel/ip_set_getport.c
+++ b/kernel/ip_set_getport.c
@@ -118,8 +118,10 @@ ip_set_get_ip_port(const struct sk_buff *skb, u8 pf, bool src, __be16 *port)
switch (pf) {
case AF_INET:
ret = ip_set_get_ip4_port(skb, src, port, &proto);
+ break;
case AF_INET6:
ret = ip_set_get_ip6_port(skb, src, port, &proto);
+ break;
default:
return false;
}