hush: remove superfluous assignment
[oweals/busybox.git] / networking / libiproute / iplink.c
index 1948efe3709879afe8458c6ccd0e080b5a6db78f..de7d6407e21da37f5829584c72c424dd76d57b5e 100644 (file)
 /* vi: set sw=4 ts=4: */
 /*
- * iplink.c            "ip link".
+ * iplink.c "ip link".
  *
- * Authors:    Alexey Kuznetsov, <kuznet@ms2.inr.ac.ru>
+ * Authors: Alexey Kuznetsov, <kuznet@ms2.inr.ac.ru>
  *
  * 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 <errno.h>
-#include <string.h>
-#include <unistd.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"
 
-/* take from linux/sockios.h */
+/* taken from linux/sockios.h */
 #define SIOCSIFNAME    0x8923          /* set interface name */
 
-static int do_link;
-
-static int on_off(char *msg)
-{
-       bb_error_msg("Error: argument of \"%s\" must be \"on\" or \"off\"", msg);
-       return -1;
-}
-
+/* Exits on error */
 static int get_ctl_fd(void)
 {
-       int s_errno;
        int fd;
 
        fd = socket(PF_INET, SOCK_DGRAM, 0);
        if (fd >= 0)
                return fd;
-       s_errno = errno;
        fd = socket(PF_PACKET, SOCK_DGRAM, 0);
        if (fd >= 0)
                return fd;
-       fd = socket(PF_INET6, SOCK_DGRAM, 0);
-       if (fd >= 0)
-               return fd;
-       errno = s_errno;
-       perror("Cannot create control socket");
-       return -1;
+       return xsocket(PF_INET6, SOCK_DGRAM, 0);
 }
 
-static int do_chflags(char *dev, __u32 flags, __u32 mask)
+/* Exits on error */
+static void do_chflags(char *dev, uint32_t flags, uint32_t mask)
 {
        struct ifreq ifr;
        int fd;
-       int err;
 
-       strcpy(ifr.ifr_name, dev);
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
        fd = get_ctl_fd();
-       if (fd < 0)
-               return -1;
-       err = ioctl(fd, SIOCGIFFLAGS, &ifr);
-       if (err) {
-               perror("SIOCGIFFLAGS");
-               close(fd);
-               return -1;
-       }
-       if ((ifr.ifr_flags^flags)&mask) {
+       xioctl(fd, SIOCGIFFLAGS, &ifr);
+       if ((ifr.ifr_flags ^ flags) & mask) {
                ifr.ifr_flags &= ~mask;
-               ifr.ifr_flags |= mask&flags;
-               err = ioctl(fd, SIOCSIFFLAGS, &ifr);
-               if (err)
-                       perror("SIOCSIFFLAGS");
+               ifr.ifr_flags |= mask & flags;
+               xioctl(fd, SIOCSIFFLAGS, &ifr);
        }
        close(fd);
-       return err;
 }
 
-static int do_changename(char *dev, char *newdev)
+/* Exits on error */
+static void do_changename(char *dev, char *newdev)
 {
        struct ifreq ifr;
        int fd;
-       int err;
 
-       strcpy(ifr.ifr_name, dev);
-       strcpy(ifr.ifr_newname, newdev);
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
+       strncpy_IFNAMSIZ(ifr.ifr_newname, newdev);
        fd = get_ctl_fd();
-       if (fd < 0)
-               return -1;
-       err = ioctl(fd, SIOCSIFNAME, &ifr);
-       if (err) {
-               perror("SIOCSIFNAME");
-               close(fd);
-               return -1;
-       }
+       xioctl(fd, SIOCSIFNAME, &ifr);
        close(fd);
-       return err;
 }
 
-static int set_qlen(char *dev, int qlen)
+/* Exits on error */
+static void set_qlen(char *dev, int qlen)
 {
        struct ifreq ifr;
        int s;
 
        s = get_ctl_fd();
-       if (s < 0)
-               return -1;
-
        memset(&ifr, 0, sizeof(ifr));
-       strcpy(ifr.ifr_name, dev);
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
        ifr.ifr_qlen = qlen;
-       if (ioctl(s, SIOCSIFTXQLEN, &ifr) < 0) {
-               perror("SIOCSIFXQLEN");
-               close(s);
-               return -1;
-       }
+       xioctl(s, SIOCSIFTXQLEN, &ifr);
        close(s);
-
-       return 0;
 }
 
-static int set_mtu(char *dev, int mtu)
+/* Exits on error */
+static void set_mtu(char *dev, int mtu)
 {
        struct ifreq ifr;
        int s;
 
        s = get_ctl_fd();
-       if (s < 0)
-               return -1;
-
        memset(&ifr, 0, sizeof(ifr));
-       strcpy(ifr.ifr_name, dev);
+       strncpy_IFNAMSIZ(ifr.ifr_name, dev);
        ifr.ifr_mtu = mtu;
-       if (ioctl(s, SIOCSIFMTU, &ifr) < 0) {
-               perror("SIOCSIFMTU");
-               close(s);
-               return -1;
-       }
+       xioctl(s, SIOCSIFMTU, &ifr);
        close(s);
-
-       return 0;
 }
 
+/* Exits on error */
 static int get_address(char *dev, int *htype)
 {
        struct ifreq ifr;
@@ -156,80 +101,71 @@ static int get_address(char *dev, int *htype)
        socklen_t alen;
        int s;
 
-       s = socket(PF_PACKET, SOCK_DGRAM, 0);
-       if (s < 0) {
-               perror("socket(PF_PACKET)");
-               return -1;
-       }
+       s = xsocket(PF_PACKET, SOCK_DGRAM, 0);
 
        memset(&ifr, 0, sizeof(ifr));
-       strcpy(ifr.ifr_name, dev);
-       if (ioctl(s, SIOCGIFINDEX, &ifr) < 0) {
-               perror("SIOCGIFINDEX");
-               close(s);
-               return -1;
-       }
+       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) {
-               perror("bind");
-               close(s);
-               return -1;
-       }
-
+       xbind(s, (struct sockaddr*)&me, sizeof(me));
        alen = sizeof(me);
-       if (getsockname(s, (struct sockaddr*)&me, &alen) == -1) {
-               perror("getsockname");
-               close(s);
-               return -1;
-       }
+       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;
 }
 
-static int parse_address(char *dev, int hatype, int halen, char *lla, struct ifreq *ifr)
+/* Exits on error */
+static void parse_address(char *dev, int hatype, int halen, char *lla, struct ifreq *ifr)
 {
        int alen;
 
        memset(ifr, 0, sizeof(*ifr));
-       strcpy(ifr->ifr_name, dev);
+       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)
-               return -1;
+               exit(EXIT_FAILURE);
        if (alen != halen) {
-               bb_error_msg("Wrong address (%s) length: expected %d bytes", lla, halen);
-               return -1;
+               bb_error_msg_and_die("wrong address (%s) length: expected %d bytes", lla, halen);
        }
-       return 0;
 }
 
-static int set_address(struct ifreq *ifr, int brd)
+/* Exits on error */
+static void set_address(struct ifreq *ifr, int brd)
 {
        int s;
 
        s = get_ctl_fd();
-       if (s < 0)
-               return -1;
-       if (ioctl(s, brd?SIOCSIFHWBROADCAST:SIOCSIFHWADDR, ifr) < 0) {
-               perror(brd?"SIOCSIFHWBROADCAST":"SIOCSIFHWADDR");
-               close(s);
-               return -1;
-       }
+       if (brd)
+               xioctl(s, SIOCSIFHWBROADCAST, ifr);
+       else
+               xioctl(s, SIOCSIFHWADDR, ifr);
        close(s);
-       return 0;
 }
 
 
-static int do_set(int argc, char **argv)
+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(char **argv)
 {
        char *dev = NULL;
-       __u32 mask = 0;
-       __u32 flags = 0;
+       uint32_t mask = 0;
+       uint32_t flags = 0;
        int qlen = -1;
        int mtu = -1;
        char *newaddr = NULL;
@@ -237,121 +173,133 @@ 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
-                               return 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
-                               return 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++;
+               argv++;
        }
 
        if (!dev) {
-               bb_error_msg(bb_msg_requires_arg, "\"dev\"");
-               exit(-1);
+               bb_error_msg_and_die(bb_msg_requires_arg, "\"dev\"");
        }
 
        if (newaddr || newbrd) {
                halen = get_address(dev, &htype);
-               if (halen < 0)
-                       return -1;
                if (newaddr) {
-                       if (parse_address(dev, htype, halen, newaddr, &ifr0) < 0)
-                               return -1;
+                       parse_address(dev, htype, halen, newaddr, &ifr0);
                }
                if (newbrd) {
-                       if (parse_address(dev, htype, halen, newbrd, &ifr1) < 0)
-                               return -1;
+                       parse_address(dev, htype, halen, newbrd, &ifr1);
                }
        }
 
        if (newname && strcmp(dev, newname)) {
-               if (do_changename(dev, newname) < 0)
-                       return -1;
+               do_changename(dev, newname);
                dev = newname;
        }
        if (qlen != -1) {
-               if (set_qlen(dev, qlen) < 0)
-                       return -1;
+               set_qlen(dev, qlen);
        }
        if (mtu != -1) {
-               if (set_mtu(dev, mtu) < 0)
-                       return -1;
+               set_mtu(dev, mtu);
        }
        if (newaddr || newbrd) {
                if (newbrd) {
-                       if (set_address(&ifr1, 1) < 0)
-                               return -1;
+                       set_address(&ifr1, 1);
                }
                if (newaddr) {
-                       if (set_address(&ifr0, 0) < 0)
-                               return -1;
+                       set_address(&ifr0, 0);
                }
        }
        if (mask)
-               return do_chflags(dev, flags, mask);
+               do_chflags(dev, flags, mask);
        return 0;
 }
 
-static int ipaddr_list_link(int argc, char **argv)
+static int ipaddr_list_link(char **argv)
 {
        preferred_family = AF_PACKET;
-       do_link = 1;
-       return ipaddr_list_or_flush(argc, argv, 0);
+       return ipaddr_list_or_flush(argv, 0);
 }
 
-int do_iplink(int argc, char **argv)
+/* Return value becomes exitcode. It's okay to not return at all */
+int do_iplink(char **argv)
 {
-       if (argc > 0) {
-               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);
-       } else
-               return ipaddr_list_link(0, NULL);
-
-       bb_error_msg("Command \"%s\" is unknown.", *argv);
-       exit(-1);
+       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);
 }