All this CONFIG_BRIDGE_NF physindev/physoutdev conditionalisation is
rather ugly to me, why did you introduce it?  If bridge-nf isn't
being used, these values should always be NULL and the code ought
to do the right thing automatically.  (In the end I want to move
away from CONFIG_BRIDGE_NF being a config option at all, just making
it conditional on CONFIG_NETFILTER or something.)


--- linux-2.4.19-brnf-0.0.7/net/core/netfilter.c        Fri Sep 13 14:56:39 2002+++ 
linux-2.4.19-brnf-0.0.8bds/net/core/netfilter.c     Fri Sep 13 15:13:26 2002@@ -418,8 
+418,10 @@
 {
        int status;
        struct nf_info *info;
+#ifdef CONFIG_BRIDGE_NF
        struct net_device *physindev;
        struct net_device *physoutdev;
+#endif

        if (!queue_handler[pf].outfn) {
                kfree_skb(skb);


_______________________________________________
Bridge mailing list
[EMAIL PROTECTED]
http://www.math.leidenuniv.nl/mailman/listinfo/bridge

Reply via email to