[IPV4] Fix ip=dhcp regression
authorSimon Horman <horms@verge.net.au>
Fri, 28 Dec 2007 05:19:10 +0000 (21:19 -0800)
committerDavid S. Miller <davem@davemloft.net>
Fri, 28 Dec 2007 21:39:11 +0000 (13:39 -0800)
David Brownell pointed out a regression in my recent "Fix ip command
line processing" patch. It turns out to be a fairly blatant oversight on
my part whereby ic_enable is never set, and thus autoconfiguration is
never enabled. Clearly my testing was broken :-(

The solution that I have is to set ic_enable to 1 if we hit
ip_auto_config_setup(), which basically means that autoconfiguration is
activated unless told otherwise. I then flip ic_enable to 0 if ip=off,
ip=none, ip=::::::off or ip=::::::none using ic_proto_name();

The incremental patch is below, let me know if a non-incremental version
is prepared, as I did as for the original patch to be reverted pending a
fix.

Signed-off-by: Simon Horman <horms@verge.net.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ipconfig.c

index 420f56ad59f87fb3b27327715de8bb4e38dad2fb..56a675734ea555ba256576a33f39cccd4c60cf91 100644 (file)
@@ -1440,6 +1440,7 @@ static int __init ip_auto_config_setup(char *addrs)
        int num = 0;
 
        ic_set_manually = 1;
+       ic_enable = 1;
 
        if (ic_proto_name(addrs))
                return 1;