hush: remove superfluous assignment
[oweals/busybox.git] / networking / libiproute / iplink.c
index 0943726e1617c185976c9515880e74a1bfedbf3e..de7d6407e21da37f5829584c72c424dd76d57b5e 100644 (file)
@@ -7,30 +7,20 @@
  * Licensed under GPLv2 or later, see file LICENSE in this tarball for details.
  */
 
-#include "libbb.h"
-
-#include <sys/ioctl.h>
-#include <sys/socket.h>
-
+//#include <sys/ioctl.h>
+//#include <sys/socket.h>
 #include <net/if.h>
 #include <net/if_packet.h>
 #include <netpacket/packet.h>
-
 #include <net/ethernet.h>
 
+#include "ip_common.h"  /* #include "libbb.h" is inside */
 #include "rt_names.h"
 #include "utils.h"
-#include "ip_common.h"
 
 /* taken from linux/sockios.h */
 #define SIOCSIFNAME    0x8923          /* set interface name */
 
-static void on_off(const char *msg) ATTRIBUTE_NORETURN;
-static void on_off(const char *msg)
-{
-       bb_error_msg_and_die("error: argument of \"%s\" must be \"on\" or \"off\"", msg);
-}
-
 /* Exits on error */
 static int get_ctl_fd(void)
 {
@@ -42,10 +32,7 @@ static int get_ctl_fd(void)
        fd = socket(PF_PACKET, SOCK_DGRAM, 0);
        if (fd >= 0)
                return fd;
-       fd = socket(PF_INET6, SOCK_DGRAM, 0);
-       if (fd >= 0)
-               return fd;
-       bb_perror_msg_and_die("cannot create control socket");
+       return xsocket(PF_INET6, SOCK_DGRAM, 0);
 }
 
 /* Exits on error */
@@ -54,16 +41,13 @@ static void do_chflags(char *dev, uint32_t flags, uint32_t mask)
        struct ifreq ifr;
        int fd;
 
-       strncpy(ifr.ifr_name, dev, sizeof(ifr.ifr_name));
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
        fd = get_ctl_fd();
-       if (ioctl(fd, SIOCGIFFLAGS, &ifr)) {
-               bb_perror_msg_and_die("SIOCGIFFLAGS");
-       }
+       xioctl(fd, SIOCGIFFLAGS, &ifr);
        if ((ifr.ifr_flags ^ flags) & mask) {
                ifr.ifr_flags &= ~mask;
                ifr.ifr_flags |= mask & flags;
-               if (ioctl(fd, SIOCSIFFLAGS, &ifr))
-                       bb_perror_msg_and_die("SIOCSIFFLAGS");
+               xioctl(fd, SIOCSIFFLAGS, &ifr);
        }
        close(fd);
 }
@@ -73,15 +57,11 @@ static void do_changename(char *dev, char *newdev)
 {
        struct ifreq ifr;
        int fd;
-       int err;
 
-       strncpy(ifr.ifr_name, dev, sizeof(ifr.ifr_name));
-       strncpy(ifr.ifr_newname, newdev, sizeof(ifr.ifr_newname));
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
+       strncpy_IFNAMSIZ(ifr.ifr_newname, newdev);
        fd = get_ctl_fd();
-       err = ioctl(fd, SIOCSIFNAME, &ifr);
-       if (err) {
-               bb_perror_msg_and_die("SIOCSIFNAME");
-       }
+       xioctl(fd, SIOCSIFNAME, &ifr);
        close(fd);
 }
 
@@ -93,11 +73,9 @@ static void set_qlen(char *dev, int qlen)
 
        s = get_ctl_fd();
        memset(&ifr, 0, sizeof(ifr));
-       strncpy(ifr.ifr_name, dev, sizeof(ifr.ifr_name));
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
        ifr.ifr_qlen = qlen;
-       if (ioctl(s, SIOCSIFTXQLEN, &ifr) < 0) {
-               bb_perror_msg_and_die("SIOCSIFXQLEN");
-       }
+       xioctl(s, SIOCSIFTXQLEN, &ifr);
        close(s);
 }
 
@@ -109,11 +87,9 @@ static void set_mtu(char *dev, int mtu)
 
        s = get_ctl_fd();
        memset(&ifr, 0, sizeof(ifr));
-       strncpy(ifr.ifr_name, dev, sizeof(ifr.ifr_name));
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
        ifr.ifr_mtu = mtu;
-       if (ioctl(s, SIOCSIFMTU, &ifr) < 0) {
-               bb_perror_msg_and_die("SIOCSIFMTU");
-       }
+       xioctl(s, SIOCSIFMTU, &ifr);
        close(s);
 }
 
@@ -125,29 +101,22 @@ static int get_address(char *dev, int *htype)
        socklen_t alen;
        int s;
 
-       s = socket(PF_PACKET, SOCK_DGRAM, 0);
-       if (s < 0) {
-               bb_perror_msg_and_die("socket(PF_PACKET)");
-       }
+       s = xsocket(PF_PACKET, SOCK_DGRAM, 0);
 
        memset(&ifr, 0, sizeof(ifr));
-       strncpy(ifr.ifr_name, dev, sizeof(ifr.ifr_name));
-       if (ioctl(s, SIOCGIFINDEX, &ifr) < 0) {
-               bb_perror_msg_and_die("SIOCGIFINDEX");
-       }
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
+       xioctl(s, SIOCGIFINDEX, &ifr);
 
        memset(&me, 0, sizeof(me));
        me.sll_family = AF_PACKET;
        me.sll_ifindex = ifr.ifr_ifindex;
        me.sll_protocol = htons(ETH_P_LOOP);
-       if (bind(s, (struct sockaddr*)&me, sizeof(me)) == -1) {
-               bb_perror_msg_and_die("bind");
-       }
-
+       xbind(s, (struct sockaddr*)&me, sizeof(me));
        alen = sizeof(me);
-       if (getsockname(s, (struct sockaddr*)&me, &alen) == -1) {
-               bb_perror_msg_and_die("getsockname");
-       }
+       getsockname(s, (struct sockaddr*)&me, &alen);
+       //never happens:
+       //if (getsockname(s, (struct sockaddr*)&me, &alen) == -1)
+       //      bb_perror_msg_and_die("getsockname");
        close(s);
        *htype = me.sll_hatype;
        return me.sll_halen;
@@ -159,11 +128,13 @@ static void parse_address(char *dev, int hatype, int halen, char *lla, struct if
        int alen;
 
        memset(ifr, 0, sizeof(*ifr));
-       strncpy(ifr->ifr_name, dev, sizeof(ifr->ifr_name));
+       strncpy_IFNAMSIZ(ifr->ifr_name, dev);
        ifr->ifr_hwaddr.sa_family = hatype;
-       alen = ll_addr_a2n((unsigned char *)(ifr->ifr_hwaddr.sa_data), 14, lla);
+
+       alen = hatype == 1/*ARPHRD_ETHER*/ ? 14/*ETH_HLEN*/ : 19/*INFINIBAND_HLEN*/;
+       alen = ll_addr_a2n((unsigned char *)(ifr->ifr_hwaddr.sa_data), alen, lla);
        if (alen < 0)
-               exit(1);
+               exit(EXIT_FAILURE);
        if (alen != halen) {
                bb_error_msg_and_die("wrong address (%s) length: expected %d bytes", lla, halen);
        }
@@ -175,15 +146,22 @@ static void set_address(struct ifreq *ifr, int brd)
        int s;
 
        s = get_ctl_fd();
-       if (ioctl(s, brd ? SIOCSIFHWBROADCAST  :SIOCSIFHWADDR, ifr) < 0) {
-               bb_perror_msg_and_die(brd ? "SIOCSIFHWBROADCAST" : "SIOCSIFHWADDR");
-       }
+       if (brd)
+               xioctl(s, SIOCSIFHWBROADCAST, ifr);
+       else
+               xioctl(s, SIOCSIFHWADDR, ifr);
        close(s);
 }
 
 
+static void die_must_be_on_off(const char *msg) NORETURN;
+static void die_must_be_on_off(const char *msg)
+{
+       bb_error_msg_and_die("argument of \"%s\" must be \"on\" or \"off\"", msg);
+}
+
 /* Return value becomes exitcode. It's okay to not return at all */
-static int do_set(int argc, char **argv)
+static int do_set(char **argv)
 {
        char *dev = NULL;
        uint32_t mask = 0;
@@ -195,53 +173,73 @@ static int do_set(int argc, char **argv)
        struct ifreq ifr0, ifr1;
        char *newname = NULL;
        int htype, halen;
-
-       while (argc > 0) {
-               if (strcmp(*argv, "up") == 0) {
+       static const char keywords[] ALIGN1 =
+               "up\0""down\0""name\0""mtu\0""multicast\0"
+               "arp\0""address\0""dev\0";
+       enum { ARG_up = 0, ARG_down, ARG_name, ARG_mtu, ARG_multicast,
+               ARG_arp, ARG_addr, ARG_dev };
+       static const char str_on_off[] ALIGN1 = "on\0""off\0";
+       enum { PARM_on = 0, PARM_off };
+       smalluint key;
+
+       while (*argv) {
+               /* substring search ensures that e.g. "addr" and "address"
+                * are both accepted */
+               key = index_in_substrings(keywords, *argv);
+               if (key == ARG_up) {
                        mask |= IFF_UP;
                        flags |= IFF_UP;
-               } else if (strcmp(*argv, "down") == 0) {
+               }
+               if (key == ARG_down) {
                        mask |= IFF_UP;
                        flags &= ~IFF_UP;
-               } else if (strcmp(*argv, "name") == 0) {
+               }
+               if (key == ARG_name) {
                        NEXT_ARG();
                        newname = *argv;
-               } else if (strcmp(*argv, "mtu") == 0) {
+               }
+               if (key == ARG_mtu) {
                        NEXT_ARG();
                        if (mtu != -1)
                                duparg("mtu", *argv);
-                       if (get_integer(&mtu, *argv, 0))
-                               invarg(*argv, "mtu");
-               } else if (strcmp(*argv, "multicast") == 0) {
+                       mtu = get_unsigned(*argv, "mtu");
+               }
+               if (key == ARG_multicast) {
+                       int param;
                        NEXT_ARG();
                        mask |= IFF_MULTICAST;
-                       if (strcmp(*argv, "on") == 0) {
+                       param = index_in_strings(str_on_off, *argv);
+                       if (param < 0)
+                               die_must_be_on_off("multicast");
+                       if (param == PARM_on)
                                flags |= IFF_MULTICAST;
-                       } else if (strcmp(*argv, "off") == 0) {
+                       else
                                flags &= ~IFF_MULTICAST;
-                       } else
-                               on_off("multicast");
-               } else if (strcmp(*argv, "arp") == 0) {
+               }
+               if (key == ARG_arp) {
+                       int param;
                        NEXT_ARG();
                        mask |= IFF_NOARP;
-                       if (strcmp(*argv, "on") == 0) {
+                       param = index_in_strings(str_on_off, *argv);
+                       if (param < 0)
+                               die_must_be_on_off("arp");
+                       if (param == PARM_on)
                                flags &= ~IFF_NOARP;
-                       } else if (strcmp(*argv, "off") == 0) {
+                       else
                                flags |= IFF_NOARP;
-                       } else
-                               on_off("noarp");
-               } else if (strcmp(*argv, "addr") == 0) {
+               }
+               if (key == ARG_addr) {
                        NEXT_ARG();
                        newaddr = *argv;
-               } else {
-                       if (strcmp(*argv, "dev") == 0) {
+               }
+               if (key >= ARG_dev) {
+                       if (key == ARG_dev) {
                                NEXT_ARG();
                        }
                        if (dev)
                                duparg2("dev", *argv);
                        dev = *argv;
                }
-               argc--;
                argv++;
        }
 
@@ -282,25 +280,26 @@ static int do_set(int argc, char **argv)
        return 0;
 }
 
-static int ipaddr_list_link(int argc, char **argv)
+static int ipaddr_list_link(char **argv)
 {
        preferred_family = AF_PACKET;
-       return ipaddr_list_or_flush(argc, argv, 0);
+       return ipaddr_list_or_flush(argv, 0);
 }
 
 /* Return value becomes exitcode. It's okay to not return at all */
-int do_iplink(int argc, char **argv)
+int do_iplink(char **argv)
 {
-       if (argc <= 0)
-               return ipaddr_list_link(0, NULL);
-
-       if (matches(*argv, "set") == 0)
-               return do_set(argc-1, argv+1);
-
-       if (matches(*argv, "show") == 0 ||
-           matches(*argv, "lst") == 0 ||
-           matches(*argv, "list") == 0)
-               return ipaddr_list_link(argc-1, argv+1);
-
-       bb_error_msg_and_die("command \"%s\" is unknown", *argv);
+       static const char keywords[] ALIGN1 =
+               "set\0""show\0""lst\0""list\0";
+       int key;
+       if (!*argv)
+               return ipaddr_list_link(argv);
+       key = index_in_substrings(keywords, *argv);
+       if (key < 0)
+               bb_error_msg_and_die(bb_msg_invalid_arg, *argv, applet_name);
+       argv++;
+       if (key == 0) /* set */
+               return do_set(argv);
+       /* show, lst, list */
+       return ipaddr_list_link(argv);
 }