Prev: Email Admin Warning Alert !!!
Next: [PATCH] rapidio: fix new kernel-doc warnings
From: Jeffrey Merkey on 28 May 2010 18:10
---------- Forwarded message ----------
From: Jeffrey Merkey <jeffmerkey(a)gmail.com>
Date: Fri, May 28, 2010 at 3:57 PM
Subject: Re: Question about SIOCGIFCONF
To: Eric Dumazet <eric.dumazet(a)gmail.com>


Not a bad suggestion, however, redhat and all the major distros still
use ifconfig. � Do you think this should be fixed or is it a
non-issue?

Jeff

On Fri, May 28, 2010 at 2:06 PM, Eric Dumazet <eric.dumazet(a)gmail.com> wrote:
> Le vendredi 28 mai 2010 � 13:05 -0600, Jeffrey Merkey a �crit :
>> The code in question is net-tools/lib/interface.c function
>> if_readproc() and associated routines. �Looks like a hole.
>>
>> Jeff
>>
>> On Fri, May 28, 2010 at 1:02 PM, Jeffrey Merkey <jeffmerkey(a)gmail.com> wrote:
>> > Review of the net-tools source code for IFCONFIG indicates that when
>> > /proc is not loaded and/or /proc/net/dev is not available, IFCONFIG
>> > will attempt to use this ioctl to determine which interfaces are
>> > present in the system. �Since the ioctl will not report unbound
>> > interfaces which are active, IFCONFIG will not properly report or
>> > detect network adapters which are unbound. �This seems to be a hole,
>> > although most of the time I assume /proc will always be mounted.
>> > Someone should review this and make a decision as to whether or not
>> > this could be a problem. �At any rate, it does not work as advertised.
>> >
>
> Maybe you could forget about a 20 years old legacy program and use the
> real thing : ip
>
> ip link
> ip addr
> ...
>
> Alternatively, you could rewrite ifconfig to use modern API.
>
> (Not depending on /proc , at all)
>
>
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
 | 
Pages: 1
Prev: Email Admin Warning Alert !!!
Next: [PATCH] rapidio: fix new kernel-doc warnings