Save a hundred bytes or so on the do_iproute function

2 files changed, 49 insertions, 53 deletions

diff --git a/networking/libiproute/ip_common.h b/networking/libiproute/ip_common.h
index 771ca48bd..0686136c7 100644
--- a/networking/libiproute/ip_common.h
+++ b/networking/libiproute/ip_common.h
@@ -9,10 +9,8 @@ extern int ipaddr_list(int argc, char **argv);
 extern int ipaddr_list_link(int argc, char **argv);
 extern int iproute_monitor(int argc, char **argv);
 extern void iplink_usage(void) __attribute__((noreturn));
-extern void iproute_reset_filter(void);
 extern void ipaddr_reset_filter(int);
 extern void ipneigh_reset_filter(void);
-extern int print_route(struct sockaddr_nl *who, struct nlmsghdr *n, void *arg);
 extern int do_ipaddr(int argc, char **argv);
 extern int do_iproute(int argc, char **argv);
 extern int do_iprule(int argc, char **argv);
diff --git a/networking/libiproute/iproute.c b/networking/libiproute/iproute.c
index b5833d7cc..cef67c34f 100644
--- a/networking/libiproute/iproute.c
+++ b/networking/libiproute/iproute.c
@@ -30,7 +30,6 @@
 
 #include "rt_names.h"
 #include "utils.h"
-#include "ip_common.h"
 
 #include "busybox.h"
 
@@ -60,7 +59,7 @@ static struct
         inet_prefix msrc;
 } filter;
 
-int print_route(struct sockaddr_nl *who, struct nlmsghdr *n, void *arg)
+static int print_route(struct sockaddr_nl *who, struct nlmsghdr *n, void *arg)
 {
         FILE *fp = (FILE*)arg;
         struct rtmsg *r = NLMSG_DATA(n);
@@ -238,7 +237,7 @@ int print_route(struct sockaddr_nl *who, struct nlmsghdr *n, void *arg)
         return 0;
 }
 
-int iproute_modify(int cmd, unsigned flags, int argc, char **argv)
+static int iproute_modify(int cmd, unsigned flags, int argc, char **argv)
 {
         struct rtnl_handle rth;
         struct {
@@ -402,6 +401,13 @@ static int rtnl_rtcache_request(struct rtnl_handle *rth, int family)
         return sendto(rth->fd, (void*)&req, sizeof(req), 0, (struct sockaddr*)&nladdr, sizeof(nladdr));
 }
 
+static void iproute_reset_filter(void)
+{
+        memset(&filter, 0, sizeof(filter));
+        filter.mdst.bitlen = -1;
+        filter.msrc.bitlen = -1;
+}
+
 static int iproute_list(int argc, char **argv)
 {
         int do_ipv6 = preferred_family;
@@ -516,7 +522,7 @@ static int iproute_list(int argc, char **argv)
 }
 
 
-int iproute_get(int argc, char **argv)
+static int iproute_get(int argc, char **argv)
 {
         struct rtnl_handle rth;
         struct {
@@ -671,54 +677,46 @@ int iproute_get(int argc, char **argv)
         exit(0);
 }
 
-void iproute_reset_filter()
-{
-        memset(&filter, 0, sizeof(filter));
-        filter.mdst.bitlen = -1;
-        filter.msrc.bitlen = -1;
-}
-
 int do_iproute(int argc, char **argv)
 {
-        if (argc < 1) {
-                return iproute_list(0, NULL);
-        }
-        
-        if (matches(*argv, "add") == 0) {
-                return iproute_modify(RTM_NEWROUTE, NLM_F_CREATE|NLM_F_EXCL,
-                                      argc-1, argv+1);
-        }
-        if (matches(*argv, "change") == 0 || strcmp(*argv, "chg") == 0) {
-                return iproute_modify(RTM_NEWROUTE, NLM_F_REPLACE,
-                                      argc-1, argv+1);
-        }
-        if (matches(*argv, "replace") == 0) {
-                return iproute_modify(RTM_NEWROUTE, NLM_F_CREATE|NLM_F_REPLACE,
-                                      argc-1, argv+1);
-        }
-        if (matches(*argv, "prepend") == 0) {
-                return iproute_modify(RTM_NEWROUTE, NLM_F_CREATE,
-                                      argc-1, argv+1);
-        }
-        if (matches(*argv, "append") == 0) {
-                return iproute_modify(RTM_NEWROUTE, NLM_F_CREATE|NLM_F_APPEND,
-                                      argc-1, argv+1);
-        }
-        if (matches(*argv, "test") == 0) {
-                return iproute_modify(RTM_NEWROUTE, NLM_F_EXCL,
-                                      argc-1, argv+1);
-        }
-        if (matches(*argv, "delete") == 0) {
-                return iproute_modify(RTM_DELROUTE, 0,
-                                      argc-1, argv+1);
-        }
-        if (matches(*argv, "list") == 0 || matches(*argv, "show") == 0
-            || matches(*argv, "lst") == 0) {
-                return iproute_list(argc-1, argv+1);
-        }
-        if (matches(*argv, "get") == 0) {
-                return iproute_get(argc-1, argv+1);
-        }
-        error_msg_and_die("Command \"%s\" is unknown, try \"ip route help\".", *argv);
+        const char *ip_route_commands[] = { "add", "append", "change", "chg",
+                "delete", "get", "list", "show", "prepend", "replace", "test", 0 };
+        unsigned short command_num = 6;
+        unsigned int flags = 0;
+        int cmd = RTM_NEWROUTE;
+
+        if (*argv) {
+                command_num = compare_string_array(ip_route_commands, *argv);
+        }
+        switch(command_num) {
+                case 0: /* add*/
+                        flags = NLM_F_CREATE|NLM_F_EXCL;
+                        break;
+                case 1: /* append */
+                        flags = NLM_F_CREATE|NLM_F_APPEND;
+                        break;
+                case 2: /* change */
+                case 3: /* chg */
+                        flags = NLM_F_REPLACE;
+                        break;
+                case 4: /* delete */
+                        cmd = RTM_DELROUTE;
+                        break;
+                case 5: /* get */
+                        return iproute_get(argc-1, argv+1);
+                case 6: /* list */
+                case 7: /* show */
+                        return iproute_list(argc-1, argv+1);
+                case 8: /* prepend */
+                        flags = NLM_F_CREATE;
+                case 9: /* replace */
+                        flags = NLM_F_CREATE|NLM_F_REPLACE;
+                case 10: /* test */
+                        flags = NLM_F_EXCL;
+                default:
+                        error_msg_and_die("Unknown command %s", *argv);
+        }
+
+        return iproute_modify(cmd, flags, argc-1, argv+1);
 }