lkml.org 
[lkml]   [2001]   [Sep]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: notion of a local address [was: Re: ioctl SIOCGIFNETMASK: ip alias
Andi Kleen did not read the patch, but flamed:

> On Thu, Sep 06, 2001 at 09:23:43PM +0100, Alan Cox wrote:
> > > Alan, SIOCGIFCONF is working sufficiently, it's SIOCGIFNETMASK that
> > > we're talking about. SIOCGIFNETMASK works properly on any other system
> > > or - as far as I can currently test - with my patch.
> >
> > Then that is worth looking into.
>
> First it would break lots of linux user land (which doesn't initialise sin_addr
> in SIOCGIFNETMASK) and special casing for "local networks" is in any case bogus.
> Netmask checking is only needed for that; it's not even needed for local address
> checking.

Sorry, Sir. You are missing the point, and it seems to me you have
neither looked at my patch nor at its description.

As a special service, I'll reiterate:

1. The new behaviour (match address and name) is only tried at all if
user space passes in an AF_INET address. If userspace passes AF_INET
and a bogus address, the lookup will fail, and 2. below will kick in.

2. If the new behaviour doesn't come up with an interface, because the
address was wrong or not member of the AF_INET family, we just do it
the old way (match just name).

If it breaks user land, show an honest example that my patch breaks. Any
objection is invalid without accompanying counter example.

(TCP/IP stack bug fingerprinting doesn't count.)

It's frustrating when people discuss patches they haven't even looked at
and then start flaming around randomly because someone wants to change
for portability. That's doing Linux a disservice.

--
Matthias Andree

O si tacuisses, poeta mansisses.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:02    [W:0.062 / U:1.312 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site