Hi,

When reading this part of your patch I wondered whether it's enough
to clear nf_debug in just two places.  Are you sure that these are
the only places we need to clear it?  Can you briefly explain?


thanks,
Lennert


--- linux-2.5.34/net/bridge/br_forward.c        Mon Sep  9 19:35:07 2002
+++ linux-2.5.34-ebtables/net/bridge/br_forward.c       Wed Sep 11 23:06:55 2002

 static void __br_deliver(struct net_bridge_port *to, struct sk_buff *skb)
 {
        skb->dev = to->dev;
+#ifdef CONFIG_NETFILTER_DEBUG
+       skb->nf_debug = 0;
+#endif
        NF_HOOK(PF_BRIDGE, NF_BR_LOCAL_OUT, skb, NULL, skb->dev,
                        __br_forward_finish);
 }


--- linux-2.5.34/net/bridge/br_input.c  Mon Sep  9 19:35:06 2002
+++ linux-2.5.34-ebtables/net/bridge/br_input.c Wed Sep 11 23:06:55 2002
 
 unsigned char bridge_ula[6] = { 0x01, 0x80, 0xc2, 0x00, 0x00, 0x00 };
 
 static int br_pass_frame_up_finish(struct sk_buff *skb)
 {
+#ifdef CONFIG_NETFILTER_DEBUG
+       skb->nf_debug = 0;
+#endif
        netif_rx(skb);
 
        return 0;
_______________________________________________
Bridge mailing list
[EMAIL PROTECTED]
http://www.math.leidenuniv.nl/mailman/listinfo/bridge

Reply via email to