netfilter: xtables: dissolve do_match function
authorJan Engelhardt <jengelh@medozas.de>
Thu, 9 Jul 2009 17:14:18 +0000 (19:14 +0200)
committerJan Engelhardt <jengelh@medozas.de>
Sun, 2 May 2010 12:13:03 +0000 (14:13 +0200)
Signed-off-by: Jan Engelhardt <jengelh@medozas.de>
net/ipv4/netfilter/ip_tables.c
net/ipv6/netfilter/ip6_tables.c

index f92818f766710da455c9dd1e0b7ab0a1c2520dc6..265cedf886606aee55a4e3e6d12f563f62eda0ce 100644 (file)
@@ -173,21 +173,6 @@ ipt_error(struct sk_buff *skb, const struct xt_target_param *par)
        return NF_DROP;
 }
 
-/* Performance critical - called for every packet */
-static inline bool
-do_match(const struct ipt_entry_match *m, const struct sk_buff *skb,
-        struct xt_match_param *par)
-{
-       par->match     = m->u.kernel.match;
-       par->matchinfo = m->data;
-
-       /* Stop iteration if it doesn't match */
-       if (!m->u.kernel.match->match(skb, par))
-               return true;
-       else
-               return false;
-}
-
 /* Performance critical */
 static inline struct ipt_entry *
 get_entry(const void *base, unsigned int offset)
@@ -379,9 +364,12 @@ ipt_do_table(struct sk_buff *skb,
                        continue;
                }
 
-               xt_ematch_foreach(ematch, e)
-                       if (do_match(ematch, skb, &mtpar) != 0)
+               xt_ematch_foreach(ematch, e) {
+                       mtpar.match     = ematch->u.kernel.match;
+                       mtpar.matchinfo = ematch->data;
+                       if (!mtpar.match->match(skb, &mtpar))
                                goto no_match;
+               }
 
                ADD_COUNTER(e->counters, ntohs(ip->tot_len), 1);
 
index 7afa117731648f9a1f584e5c9966f6824e3634c6..f8ac4a0b58999dca9b4536feabec97a31883861d 100644 (file)
@@ -205,21 +205,6 @@ ip6t_error(struct sk_buff *skb, const struct xt_target_param *par)
        return NF_DROP;
 }
 
-/* Performance critical - called for every packet */
-static inline bool
-do_match(const struct ip6t_entry_match *m, const struct sk_buff *skb,
-        struct xt_match_param *par)
-{
-       par->match     = m->u.kernel.match;
-       par->matchinfo = m->data;
-
-       /* Stop iteration if it doesn't match */
-       if (!m->u.kernel.match->match(skb, par))
-               return true;
-       else
-               return false;
-}
-
 static inline struct ip6t_entry *
 get_entry(const void *base, unsigned int offset)
 {
@@ -402,9 +387,12 @@ ip6t_do_table(struct sk_buff *skb,
                        continue;
                }
 
-               xt_ematch_foreach(ematch, e)
-                       if (do_match(ematch, skb, &mtpar) != 0)
+               xt_ematch_foreach(ematch, e) {
+                       mtpar.match     = ematch->u.kernel.match;
+                       mtpar.matchinfo = ematch->data;
+                       if (!mtpar.match->match(skb, &mtpar))
                                goto no_match;
+               }
 
                ADD_COUNTER(e->counters,
                            ntohs(ipv6_hdr(skb)->payload_len) +