Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=03dafbbdf8a5e8a9e3e347a393930ed49506d00b
Commit:     03dafbbdf8a5e8a9e3e347a393930ed49506d00b
Parent:     06e1374a7ed45f1788353a2944a20133adc55649
Author:     Patrick McHardy <[EMAIL PROTECTED]>
AuthorDate: Mon Dec 17 21:55:16 2007 -0800
Committer:  David S. Miller <[EMAIL PROTECTED]>
CommitDate: Mon Jan 28 14:58:43 2008 -0800

    [NETFILTER]: arp_tables: remove obsolete standard_check function
    
    The size check is already performed by xt_check_target, no need
    to do it again.
    
    Signed-off-by: Patrick McHardy <[EMAIL PROTECTED]>
    Signed-off-by: David S. Miller <[EMAIL PROTECTED]>
---
 net/ipv4/netfilter/arp_tables.c |   28 +++-------------------------
 1 files changed, 3 insertions(+), 25 deletions(-)

diff --git a/net/ipv4/netfilter/arp_tables.c b/net/ipv4/netfilter/arp_tables.c
index d5cae7e..8b1842b 100644
--- a/net/ipv4/netfilter/arp_tables.c
+++ b/net/ipv4/netfilter/arp_tables.c
@@ -435,23 +435,6 @@ static int mark_source_chains(struct xt_table_info 
*newinfo,
        return 1;
 }
 
-static inline int standard_check(const struct arpt_entry_target *t,
-                                unsigned int max_offset)
-{
-       /* Check standard info. */
-       if (t->u.target_size
-           != ARPT_ALIGN(sizeof(struct arpt_standard_target))) {
-               duprintf("arpt_standard_check: target size %u != %Zu\n",
-                        t->u.target_size,
-                        ARPT_ALIGN(sizeof(struct arpt_standard_target)));
-               return 0;
-       }
-
-       return 1;
-}
-
-static struct arpt_target arpt_standard_target;
-
 static inline int check_entry(struct arpt_entry *e, const char *name, unsigned 
int size,
                              unsigned int *i)
 {
@@ -486,14 +469,9 @@ static inline int check_entry(struct arpt_entry *e, const 
char *name, unsigned i
        if (ret)
                goto err;
 
-       if (t->u.kernel.target == &arpt_standard_target) {
-               if (!standard_check(t, size)) {
-                       ret = -EINVAL;
-                       goto err;
-               }
-       } else if (t->u.kernel.target->checkentry
-                  && !t->u.kernel.target->checkentry(name, e, target, t->data,
-                                                     e->comefrom)) {
+       if (t->u.kernel.target->checkentry
+           && !t->u.kernel.target->checkentry(name, e, target, t->data,
+                                              e->comefrom)) {
                duprintf("arp_tables: check failed for `%s'.\n",
                         t->u.kernel.target->name);
                ret = -EINVAL;
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to