odhcpd: fix strncpy bounds
authorHans Dedecker <dedeckeh@gmail.com>
Thu, 3 May 2018 15:21:15 +0000 (17:21 +0200)
committerHans Dedecker <dedeckeh@gmail.com>
Thu, 3 May 2018 15:21:15 +0000 (17:21 +0200)
Fix strncpy bounds as reported by Coverity in CID 1412278 and 1412293

Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>
src/dhcpv4.c
src/odhcpd.c

index db5f138d95928f05116917d0243f09b997961d1f..35a3ad0b350b91e31e807fbcfad1d68dbf281876 100644 (file)
@@ -779,8 +779,10 @@ static void handle_dhcpv4(void *addr, void *data, size_t len,
                }
        }
 
-       struct ifreq ifr = {.ifr_name = ""};
-       strncpy(ifr.ifr_name, iface->ifname, sizeof(ifr.ifr_name));
+       struct ifreq ifr;
+
+       memset(&ifr, 0, sizeof(ifr));
+       strncpy(ifr.ifr_name, iface->ifname, sizeof(ifr.ifr_name) - 1);
 
        if (!ioctl(sock, SIOCGIFMTU, &ifr)) {
                uint16_t mtu = htons(ifr.ifr_mtu);
index 71b803d01534935037d89c9f6da9dc91f58732d1..8aa4571ea642dc15409f00f235ad8b16900e92c2 100644 (file)
@@ -143,10 +143,12 @@ int odhcpd_get_interface_config(const char *ifname, const char *what)
 int odhcpd_get_mac(const struct interface *iface, uint8_t mac[6])
 {
        struct ifreq ifr;
+
        memset(&ifr, 0, sizeof(ifr));
-       strncpy(ifr.ifr_name, iface->ifname, sizeof(ifr.ifr_name));
+       strncpy(ifr.ifr_name, iface->ifname, sizeof(ifr.ifr_name) - 1);
        if (ioctl(ioctl_sock, SIOCGIFHWADDR, &ifr) < 0)
                return -1;
+
        memcpy(mac, ifr.ifr_hwaddr.sa_data, 6);
        return 0;
 }