vconfig: remove /proc/net/vlan/config check
authorPaulius Zaleckas <paulius.zaleckas@gmail.com>
Fri, 16 Dec 2011 00:42:55 +0000 (01:42 +0100)
committerDenys Vlasenko <vda.linux@googlemail.com>
Fri, 16 Dec 2011 00:42:55 +0000 (01:42 +0100)
In original implementation /proc/net/vlan/config check only prints warning
and does not exit. Busybox version exits if this file is not found and this
prevents kernel module autoloading to work.
I think it is safe to remove this check since ioctl() call will fail
(with proper
error code) anyway if 802.1q module is not loaded (or autoloading is off).

Signed-off-by: Paulius Zaleckas <paulius.zaleckas@gmail.com>
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
networking/vconfig.c

index 48b45d9af19fd084ddbcb997cc9a64811cae9d98..ac8b96d8c519f840c2fb63d2a6b182b4f068eafd 100644 (file)
@@ -116,8 +116,6 @@ static const char name_types[] ALIGN1 = {
        '_', 'N', 'O', '_', 'P', 'A', 'D', 0,
 };
 
-static const char conf_file_name[] ALIGN1 = "/proc/net/vlan/config";
-
 int vconfig_main(int argc, char **argv) MAIN_EXTERNALLY_VISIBLE;
 int vconfig_main(int argc, char **argv)
 {
@@ -129,10 +127,6 @@ int vconfig_main(int argc, char **argv)
                bb_show_usage();
        }
 
-       /* Don't bother closing the filedes.  It will be closed on cleanup. */
-       /* Will die if 802.1q is not present */
-       xopen(conf_file_name, O_RDONLY);
-
        memset(&ifr, 0, sizeof(ifr));
 
        ++argv;