ip: make numeric table work: "ip route list table 255". closes bug 3664.

[oweals/busybox.git] / networking / libiproute / ip_parse_common_args.c

diff --git a/networking/libiproute/ip_parse_common_args.c b/networking/libiproute/ip_parse_common_args.c
index 6efbc6912031c3ff7da31696d8dd8abc0ebe3b55..5e4012b81e9999a70cb89c70b9c8c13c4522bc5a 100644 (file)
--- a/networking/libiproute/ip_parse_common_args.c
+++ b/networking/libiproute/ip_parse_common_args.c
@@ -15,65 +15,70 @@
  * Rani Assaf <rani@magic.metawire.com> 980929:        resolve addresses
  */
 
-#include <string.h>
-
-#include "libbb.h"
+#include "ip_common.h"  /* #include "libbb.h" is inside */
 #include "utils.h"
-#include "ip_common.h"
-
 
-int preferred_family = AF_UNSPEC;
+family_t preferred_family = AF_UNSPEC;
 smallint oneline;
 char _SL_;
 
-void ip_parse_common_args(int *argcp, char ***argvp)
+char **ip_parse_common_args(char **argv)
 {
-       int argc = *argcp;
-       char **argv = *argvp;
+       static const char ip_common_commands[] ALIGN1 =
+               "oneline" "\0"
+               "family" "\0"
+               "4" "\0"
+               "6" "\0"
+               "0" "\0"
+               ;
+       enum {
+               ARG_oneline,
+               ARG_family,
+               ARG_IPv4,
+               ARG_IPv6,
+               ARG_packet,
+       };
+       static const family_t af_numbers[] = { AF_INET, AF_INET6, AF_PACKET };
+       int arg;
 
-       while (argc > 1) {
-               char *opt = argv[1];
-
-               if (strcmp(opt,"--") == 0) {
-                       argc--;
-                       argv++;
-                       break;
-               }
+       while (*argv) {
+               char *opt = *argv;
 
                if (opt[0] != '-')
                        break;
-
-               if (opt[1] == '-')
+               opt++;
+               if (opt[0] == '-') {
                        opt++;
-
-               if (matches(opt, "-family") == 0) {
-                       argc--;
+                       if (!opt[0]) { /* "--" */
+                               argv++;
+                               break;
+                       }
+               }
+               arg = index_in_substrings(ip_common_commands, opt);
+               if (arg < 0)
+                       bb_show_usage();
+               if (arg == ARG_oneline) {
+                       oneline = 1;
                        argv++;
-                       if (!argv[1])
+                       continue;
+               }
+               if (arg == ARG_family) {
+                       static const char families[] ALIGN1 =
+                               "inet" "\0" "inet6" "\0" "link" "\0";
+                       argv++;
+                       if (!*argv)
                                bb_show_usage();
-                       if (strcmp(argv[1], "inet") == 0)
-                               preferred_family = AF_INET;
-                       else if (strcmp(argv[1], "inet6") == 0)
-                               preferred_family = AF_INET6;
-                       else if (strcmp(argv[1], "link") == 0)
-                               preferred_family = AF_PACKET;
-                       else
-                               invarg(argv[1], "protocol family");
-               } else if (strcmp(opt, "-4") == 0) {
-                       preferred_family = AF_INET;
-               } else if (strcmp(opt, "-6") == 0) {
-                       preferred_family = AF_INET6;
-               } else if (strcmp(opt, "-0") == 0) {
-                       preferred_family = AF_PACKET;
-               } else if (matches(opt, "-oneline") == 0) {
-                       ++oneline;
+                       arg = index_in_strings(families, *argv);
+                       if (arg < 0)
+                               invarg(*argv, "protocol family");
+                       /* now arg == 0, 1 or 2 */
                } else {
-                       bb_show_usage();
+                       arg -= ARG_IPv4;
+                       /* now arg == 0, 1 or 2 */
                }
-               argc--;
+               preferred_family = af_numbers[arg];
                argv++;
        }
-       _SL_ = oneline ? '\\' : '\n' ;
-       *argcp = argc;
-       *argvp = argv;
+       _SL_ = oneline ? '\\' : '\n';
+       return argv;
 }