netfilter: ipset: add missing break statemtns in ip_set_get_ip_port()
authorPatrick McHardy <kaber@trash.net>
Wed, 2 Feb 2011 08:31:37 +0000 (09:31 +0100)
committerPatrick McHardy <kaber@trash.net>
Wed, 2 Feb 2011 08:31:37 +0000 (09:31 +0100)
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>
net/netfilter/ipset/ip_set_getport.c

index 76737bb..4dd2785 100644 (file)
@@ -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;
        }