From 8e82fca1cd7e2a39a4e8bdbfadc73b95178be1d9 Mon Sep 17 00:00:00 2001 From: Colin Davidson Date: Mon, 25 Feb 2019 00:21:55 -0500 Subject: arp: inline get_hw_add and minor sscanf error handling cleanup --- toys/pending/arp.c | 37 ++++++++++++++++--------------------- 1 file changed, 16 insertions(+), 21 deletions(-) diff --git a/toys/pending/arp.c b/toys/pending/arp.c index 6b57c3ca..7e7b8668 100644 --- a/toys/pending/arp.c +++ b/toys/pending/arp.c @@ -84,25 +84,6 @@ static int get_index(struct type arr[], char *name) return arr[i].val; } - -void get_hw_add(char *hw_addr, char *ptr) -{ - char *p = ptr, *hw = hw_addr; - - while (*hw_addr && (p-ptr) < 6) { - int val, len = 0; - - if (*hw_addr == ':') hw_addr++; - sscanf(hw_addr, "%2x%n", &val, &len); - if (!len || len > 2) break; - hw_addr += len; - *p++ = val; - } - - if ((p-ptr) != 6 || *hw_addr) - error_exit("bad hw addr '%s'", hw); -} - static void resolve_host(char *host, struct sockaddr *sa) { struct addrinfo hints, *res = NULL; @@ -161,8 +142,22 @@ static int set_entry(void) if (!toys.optargs[1]) error_exit("bad syntax"); - if (!(toys.optflags & FLAG_D)) get_hw_add(toys.optargs[1], (char*)&req.arp_ha.sa_data); - else { + if (!(toys.optflags & FLAG_D)) { + char *ptr = toys.optargs[1]; + char *p = ptr, *hw_addr = req.arp_ha.sa_data; + + while (*hw_addr && (p-ptr) < 6) { + int val, len; + + if (*hw_addr == ':') hw_addr++; + if (!sscanf(hw_addr, "%2x%n", &val, &len)) break; + hw_addr += len; + *p++ = val; + } + + if ((p-ptr) != 6 || *hw_addr) + error_exit("bad hw addr '%s'", req.arp_ha.sa_data); + } else { struct ifreq ifre; xstrncpy(ifre.ifr_name, toys.optargs[1], IFNAMSIZ); -- cgit v1.2.3