Let commands handle themselves absence of arguments

[ngadmin] / cli / com_netconf.c

diff --git a/cli/com_netconf.c b/cli/com_netconf.c
index 5c165ae186969ddcf3720189b9699164e8eecbeb..99cd6baa9926b046de174619c499dd0afb15a5f8 100644 (file)
--- a/cli/com_netconf.c
+++ b/cli/com_netconf.c
@@ -2,7 +2,7 @@
 #include "commands.h"
 
 
-bool do_netconf_set (int nb, const char **com, struct ngadmin *nga)
+bool do_netconf_set (int argc, const char **argv, struct ngadmin *nga)
 {
        int i, k;
        const struct swi_attr *sa;
@@ -10,8 +10,8 @@ bool do_netconf_set (int nb, const char **com, struct ngadmin *nga)
        bool ret = true;
        
        
-       if (nb == 0) {
-               printf("Usage: netconf set [dhcp yes|no] [ip <ip>] [mask <mask>] [gw <gw>]\n");
+       if (argc == 0) {
+               printf("usage: netconf set [dhcp yes|no] [ip <ip>] [mask <mask>] [gw <gw>]\n");
                return false;
        }
        
@@ -23,32 +23,32 @@ bool do_netconf_set (int nb, const char **com, struct ngadmin *nga)
        
        memset(&nc, 0, sizeof(struct net_conf));
        
-       for (k = 0; k < nb; k += 2) {
-               if (strcasecmp(com[k], "dhcp") == 0) {
-                       if (strcasecmp(com[k+1], "yes") == 0) {
+       for (k = 0; k < argc; k += 2) {
+               if (strcasecmp(argv[k], "dhcp") == 0) {
+                       if (strcasecmp(argv[k + 1], "yes") == 0) {
                                nc.dhcp = true;
-                       } else if (strcasecmp(com[k+1], "no") == 0) {
+                       } else if (strcasecmp(argv[k + 1], "no") == 0) {
                                nc.dhcp = false;
                        } else {
                                printf("Incorrect DHCP value\n");
                                ret = false;
                                goto end;
                        }
-               } else if (strcasecmp(com[k], "ip") == 0) {
-                       if (inet_aton(com[k+1], &nc.ip) == 0) {
+               } else if (strcasecmp(argv[k], "ip") == 0) {
+                       if (inet_aton(argv[k + 1], &nc.ip) == 0) {
                                printf("Incorrect IP value\n");
                                ret = false;
                                goto end;
                        }
-               } else if (strcasecmp(com[k], "mask") == 0) {
+               } else if (strcasecmp(argv[k], "mask") == 0) {
                        /* TODO: check if it is a correct mask */
-                       if (inet_aton(com[k+1], &nc.netmask) == 0) {
+                       if (inet_aton(argv[k + 1], &nc.netmask) == 0) {
                                printf("Incorrect mask value\n");
                                ret = false;
                                goto end;
                        }
-               } else if (strcasecmp(com[k], "gw") == 0) {
-                       if (inet_aton(com[k+1], &nc.gw) == 0) {
+               } else if (strcasecmp(argv[k], "gw") == 0) {
+                       if (inet_aton(argv[k + 1], &nc.gw) == 0) {
                                printf("Incorrect gateway value\n");
                                ret = false;
                                goto end;