[BusyBox 0000980]: patch to avoid "broadcast +" syntax

[Denis Vlasenko]

On Wednesday 22 November 2006 09:45, Roberto A. Foglietta wrote:
> BusyBox v1.1.2 (2006.11.21-14:23+0000) Built-in shell (ash)
> Enter 'help' for a list of built-in commands.
> 
> ### original
> # ifconfig
> eth0      Link encap:Ethernet  HWaddr 00:0A:35:00:22:00
>           inet addr:172.16.241.217  Bcast:172.16.255.255  Mask:255.255.0.0
> 
> ### changing IP
> # ifconfig eth0 172.16.241.216
> # ifconfig eth0
> eth0      Link encap:Ethernet  HWaddr 00:0A:35:00:22:00
>           inet addr:172.16.241.216  Bcast:172.16.255.255  Mask:255.255.0.0
> 
> ### changing IP and netmask as you can see broadcast is not anymore
> consistent with the new netmask
> # ifconfig eth0 172.16.241.216 netmask 255.255.255.0
> # ifconfig eth0
> eth0      Link encap:Ethernet  HWaddr 00:0A:35:00:22:00
>           inet addr:172.16.241.216  Bcast:172.16.255.255  Mask:255.255.255.0

Last step doesn't work like that for me with v1.2.2.svn (2006-11-20 00:46:58 CET)

bash-3.2# ip a f dev if
bash-3.2# ifconfig if 172.16.241.217 netmask 255.255.0.0
bash-3.2# ip a l dev if
2: if: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen 1000
    link/ether 00:16:17:6d:77:db brd ff:ff:ff:ff:ff:ff
    inet 172.16.241.217/16 brd 172.16.255.255 scope global if

bash-3.2# ifconfig if 172.16.241.216
bash-3.2# ip a l dev if
2: if: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen 1000
    link/ether 00:16:17:6d:77:db brd ff:ff:ff:ff:ff:ff
    inet 172.16.241.216/16 brd 172.16.255.255 scope global if

bash-3.2# ifconfig if 172.16.241.216 netmask 255.255.255.0
bash-3.2# ip a l dev if
2: if: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen 1000
    link/ether 00:16:17:6d:77:db brd ff:ff:ff:ff:ff:ff
    inet 172.16.241.216/24 brd 172.16.241.255 scope global if

See? On last step I've got correct broadcast address configured,
without "broadcast +".

The same happens when I use "standard" (non-busyboxed) ifconfig.

In other words: I do not see the problem yet. At least in current svn.
--
vda