netdev
[Top] [All Lists]

Re: ipt_physdev.c alignment problems on parisc64

To: Harald Welte <laforge@xxxxxxxxxxxxx>
Subject: Re: ipt_physdev.c alignment problems on parisc64
From: jamal <hadi@xxxxxxxxxx>
Date: 11 Sep 2003 21:40:46 -0400
Cc: "David S. Miller" <davem@xxxxxxxxxx>, Bart De Schuymer <bdschuym@xxxxxxxxxx>, acme@xxxxxxxxxxxxxxxx, netfilter-devel@xxxxxxxxxxxxxxxxxxx, netdev@xxxxxxxxxxx
In-reply-to: <20030905153152.GA3623@xxxxxxxxxxxxxxxxxxxxxxxxxx>
Organization: jamalopolis
References: <200309022116.41697.bdschuym@xxxxxxxxxx> <20030903200426.59d0af4a.davem@xxxxxxxxxx> <20030905153152.GA3623@xxxxxxxxxxxxxxxxxxxxxxxxxx>
Reply-to: hadi@xxxxxxxxxx
Sender: netdev-bounce@xxxxxxxxxxx
Harald,

Could you not resolve all the ifindices of the said ppp+ interfaces
at rule installation time and do an integer compare instead?

cheers,
jamal

On Fri, 2003-09-05 at 11:31, Harald Welte wrote:

> > 
> > Please, just remove the super-silly memcmp() optimization in
> > the ipt_physdev.c code.
> 
> Dave, the respective code in ipt_physdev.c seems to be copied from
> the ip_tables.c interface name match (which definitely has the same
> alignment issues, btw).
> 
> The problem is that it is _not_ a simple reimplementation of memcmp(),
> but a mask-compare.
> 
> People can do stuff like "-i ppp+", meaning that traffic from all
> interfaces called "ppp<WHATEVER>" are matched.


<Prev in Thread] Current Thread [Next in Thread>