linux: improve some strscy() calls
authorFerry Huberts <ferry.huberts@pelagic.nl>
Thu, 2 Jun 2016 18:58:27 +0000 (20:58 +0200)
committerFerry Huberts <ferry.huberts@pelagic.nl>
Thu, 2 Jun 2016 20:06:48 +0000 (22:06 +0200)
Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
src/linux/kernel_tunnel.c
src/linux/net.c

index 1ed3b13..0e1bc2c 100644 (file)
@@ -156,7 +156,7 @@ int os_ip_tunnel(const char *name, void *target) {
                if (target) {
                        p4.iph.daddr = *((in_addr_t *) target);
                }
-               strscpy(p4.name, name, IFNAMSIZ);
+               strscpy(p4.name, name, sizeof(p4.name));
        } else {
 #ifdef LINUX_IPV6_TUNNEL
                p = (void *) &p6;
@@ -167,13 +167,13 @@ int os_ip_tunnel(const char *name, void *target) {
                if (target) {
                        p6.raddr = *((struct in6_addr *) target);
                }
-               strscpy(p6.name, name, IFNAMSIZ);
+               strscpy(p6.name, name, sizeof(p6.name));
 #else /* LINUX_IPV6_TUNNEL */
                return 0;
 #endif /* LINUX_IPV6_TUNNEL */
        }
 
-       strscpy(ifr.ifr_name, target != NULL ? tunName : name, IFNAMSIZ);
+       strscpy(ifr.ifr_name, target != NULL ? tunName : name, sizeof(ifr.ifr_name));
        ifr.ifr_ifru.ifru_data = p;
 
        if ((err = ioctl(olsr_cnf->ioctl_s, target != NULL ? SIOCADDTUNNEL : SIOCDELTUNNEL, &ifr))) {
@@ -225,7 +225,7 @@ struct olsr_iptunnel_entry *olsr_os_add_ipip_tunnel(union olsr_ip_addr *target,
     memcpy(&t->target, target, sizeof(*target));
     t->node.key = &t->target;
 
-    strscpy(t->if_name, name, IFNAMSIZ);
+    strscpy(t->if_name, name, sizeof(t->if_name));
     t->if_index = if_idx;
 
     avl_insert(&tunnel_tree, &t->node, AVL_DUP_NO);
index 75300af..22a0fa1 100644 (file)
@@ -717,7 +717,7 @@ bool olsr_if_isup(const char * dev)
   }
 
   memset(&ifr, 0, sizeof(ifr));
-  strscpy(ifr.ifr_name, dev, IFNAMSIZ);
+  strscpy(ifr.ifr_name, dev, sizeof(ifr.ifr_name));
 
   if (ioctl(olsr_cnf->ioctl_s, SIOCGIFFLAGS, &ifr) < 0) {
     OLSR_PRINTF(1, "ioctl SIOCGIFFLAGS (get flags) error on device %s: %s (%d)\n",
@@ -732,7 +732,7 @@ int olsr_if_set_state(const char *dev, bool up) {
   struct ifreq ifr;
 
   memset(&ifr, 0, sizeof(ifr));
-  strscpy(ifr.ifr_name, dev, IFNAMSIZ);
+  strscpy(ifr.ifr_name, dev, sizeof(ifr.ifr_name));
 
   if (ioctl(olsr_cnf->ioctl_s, SIOCGIFFLAGS, &ifr) < 0) {
     OLSR_PRINTF(1, "ioctl SIOCGIFFLAGS (get flags) error on device %s: %s (%d)\n",