On Thu, Jun 06, 2002 at 02:24:05PM +0200, Martin Josefsson wrote: > There exists code to disable tracking of broadcasts but it's insice a > #if 0 / #endif in ip_conntrack_core.c and looks like some old debugging > code as it has printk's and stuff. > > this patch removes that chunk and replaces it with a check so if the > packet is a broadcast or multicast we just return NF_ACCEPT since > there's not much point in us trying to track it.
hm. I would generally agree with you, but there is one issue which needs to be looked into before considering this patch: What happens if some client sends an ip-unicast diagram as link layer broadcast? How does the linux stack react to this? If the packet is processed/routed the same way like any other ip-unicast link-layer unicast packet, we still need to do tracking of broadcast packets. Could you try to look into that and keep us posted? Thanks. > /Martin -- Live long and prosper - Harald Welte / [EMAIL PROTECTED] http://www.gnumonks.org/ ============================================================================ GCS/E/IT d- s-: a-- C+++ UL++++$ P+++ L++++$ E--- W- N++ o? K- w--- O- M- V-- PS+ PE-- Y+ PGP++ t++ 5-- !X !R tv-- b+++ DI? !D G+ e* h+ r% y+(*)