On Sun, Feb 12, 2006 at 11:15:42PM +0100, Bernard Pidoux wrote: > Hi, > > I would like to report that when compiling 2.6.15.4 kernel > there is an error with undefined symbol 'has_bridge_parent' > in compilation of bridge/br_netfilter.c > due to CONFIG_NETFILTER_DEBUG 1 option.
I think that the following patch will resolve this problem. -- Horms [BRIDGE]: netfilter missing symbol has_bridge_parent 5dce971acf2ae20c80d5e9d1f6bbf17376870911 in Linus' tree, otherwise known as bridge-netfilter-races-on-device-removal.patch in 2.5.15.4 removed has_bridge_parent, however this symbol is still called with NETFILTER_DEBUG is enabled. This patch uses the already seeded realoutdev value to detect if a parent exists, and if so, the value of the parent. Signed-Off-By: Horms <[EMAIL PROTECTED]> diff --git a/net/bridge/br_netfilter.c b/net/bridge/br_netfilter.c index b501816..6bb0c7e 100644 --- a/net/bridge/br_netfilter.c +++ b/net/bridge/br_netfilter.c @@ -805,8 +805,8 @@ static unsigned int br_nf_post_routing(u print_error: if (skb->dev != NULL) { printk("[%s]", skb->dev->name); - if (has_bridge_parent(skb->dev)) - printk("[%s]", bridge_parent(skb->dev)->name); + if (realoutdev) + printk("[%s]", realoutdev->name); } printk(" head:%p, raw:%p, data:%p\n", skb->head, skb->mac.raw, skb->data); - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html