[PATCH] some more detail to error message about match/target mask mismatch

Balazs Scheidler bazsi at balabit.hu
Fri Jul 6 09:04:17 CEST 2007


Hi,

The error message about an invalid hook_mask in x_tables contained to
little detail (for me), so I added some more information. It's not of
utter importance to integrate it, but it helped for me, it might help
for others. With this mail, I'm dropping this patch from my patch queue.

Please apply, if you want. Thanks.

Signed-of-by: Balazs Scheidler <bazsi at balabit.hu>

diff --git a/net/netfilter/x_tables.c b/net/netfilter/x_tables.c
index 99293c6..e406300 100644
--- a/net/netfilter/x_tables.c
+++ b/net/netfilter/x_tables.c
@@ -260,8 +260,8 @@ int xt_check_match(const struct xt_match *match, unsigned short family,
                return -EINVAL;
        }
        if (match->hooks && (hook_mask & ~match->hooks) != 0) {
-               printk("%s_tables: %s match: bad hook_mask %u\n",
-                      xt_prefix[family], match->name, hook_mask);
+               printk("%s_tables: %s match: bad hook_mask %u/%u\n",
+                      xt_prefix[family], match->name, hook_mask, match->hooks);
                return -EINVAL;
        }
        if (match->proto && (match->proto != proto || inv_proto)) {
@@ -337,8 +337,8 @@ int xt_check_target(const struct xt_target *target, unsigned short family,
                return -EINVAL;
        }
        if (target->hooks && (hook_mask & ~target->hooks) != 0) {
-               printk("%s_tables: %s target: bad hook_mask %u\n",
-                      xt_prefix[family], target->name, hook_mask);
+               printk("%s_tables: %s target: bad hook_mask %u/%u\n",
+                      xt_prefix[family], target->name, hook_mask, target->hooks);
                return -EINVAL;
        }
        if (target->proto && (target->proto != proto || inv_proto)) {


-- 
Bazsi




More information about the netfilter-devel mailing list