Move IP address parsing functions to utils.

This is a patch series against the netifd repository.

This first patch moves some previously static functions to utils, so that the
follow up patch may use them from system-linux.c.

Signed-off-by: Stéphan Kochen <stephan@kochen.nl>

diff --git a/proto.c b/proto.c
index ccc5f25a89be4dce52a8c585448af602b37bc740..c72a00572cc4c8de44d6f9995be42d4e6d1267da 100644 (file)
--- a/proto.c
+++ b/proto.c
@@ -70,66 +70,6 @@ static const struct blobmsg_policy proto_ip_addr[__ADDR_MAX] = {
        [ADDR_PTP] = { .name = "ptp", .type = BLOBMSG_TYPE_STRING },
 };
 
-unsigned int
-parse_netmask_string(const char *str, bool v6)
-{
-       struct in_addr addr;
-       unsigned int ret;
-       char *err = NULL;
-
-       if (!strchr(str, '.')) {
-               ret = strtoul(str, &err, 0);
-               if (err && *err)
-                       goto error;
-
-               return ret;
-       }
-
-       if (v6)
-               goto error;
-
-       if (inet_aton(str, &addr) != 1)
-               goto error;
-
-       return 32 - fls(~(ntohl(addr.s_addr)));
-
-error:
-       return ~0;
-}
-
-static bool
-split_netmask(char *str, unsigned int *netmask, bool v6)
-{
-       char *delim = strchr(str, '/');
-
-       if (delim) {
-               *(delim++) = 0;
-
-               *netmask = parse_netmask_string(delim, v6);
-       }
-       return true;
-}
-
-static int
-parse_ip_and_netmask(int af, const char *str, void *addr, unsigned int *netmask)
-{
-       char *astr = alloca(strlen(str) + 1);
-
-       strcpy(astr, str);
-       if (!split_netmask(astr, netmask, af == AF_INET6))
-               return 0;
-
-       if (af == AF_INET6) {
-               if (*netmask > 128)
-                       return 0;
-       } else {
-               if (*netmask > 32)
-                       return 0;
-       }
-
-       return inet_pton(af, astr, addr);
-}
-
 static struct device_addr *
 alloc_device_addr(bool v6, bool ext)
 {

diff --git a/utils.c b/utils.c
index 65109d7b262ef82837dbfb2912922c4f036a3956..986cee96e147f062f24bb03a527716d71b57d8c3 100644 (file)
--- a/utils.c
+++ b/utils.c
@@ -15,6 +15,9 @@
 #include <stdlib.h>
 #include "utils.h"
 
+#include <arpa/inet.h>
+#include <netinet/in.h>
+
 void
 __vlist_simple_init(struct vlist_simple_tree *tree, int offset)
 {
@@ -66,3 +69,63 @@ vlist_simple_flush_all(struct vlist_simple_tree *tree)
        tree->version = -1;
        vlist_simple_flush(tree);
 }
+
+unsigned int
+parse_netmask_string(const char *str, bool v6)
+{
+       struct in_addr addr;
+       unsigned int ret;
+       char *err = NULL;
+
+       if (!strchr(str, '.')) {
+               ret = strtoul(str, &err, 0);
+               if (err && *err)
+                       goto error;
+
+               return ret;
+       }
+
+       if (v6)
+               goto error;
+
+       if (inet_aton(str, &addr) != 1)
+               goto error;
+
+       return 32 - fls(~(ntohl(addr.s_addr)));
+
+error:
+       return ~0;
+}
+
+bool
+split_netmask(char *str, unsigned int *netmask, bool v6)
+{
+       char *delim = strchr(str, '/');
+
+       if (delim) {
+               *(delim++) = 0;
+
+               *netmask = parse_netmask_string(delim, v6);
+       }
+       return true;
+}
+
+int
+parse_ip_and_netmask(int af, const char *str, void *addr, unsigned int *netmask)
+{
+       char *astr = alloca(strlen(str) + 1);
+
+       strcpy(astr, str);
+       if (!split_netmask(astr, netmask, af == AF_INET6))
+               return 0;
+
+       if (af == AF_INET6) {
+               if (*netmask > 128)
+                       return 0;
+       } else {
+               if (*netmask > 32)
+                       return 0;
+       }
+
+       return inet_pton(af, astr, addr);
+}

diff --git a/utils.h b/utils.h
index 319e53058329af59c84d0fab992cfc016dd87336..ec2231331c9d1cdbe4828b95f238e1268ffb7ca4 100644 (file)
--- a/utils.h
+++ b/utils.h
@@ -110,4 +110,8 @@ static inline int fls(int x)
 }
 #endif
 
+unsigned int parse_netmask_string(const char *str, bool v6);
+bool split_netmask(char *str, unsigned int *netmask, bool v6);
+int parse_ip_and_netmask(int af, const char *str, void *addr, unsigned int *netmask);
+
 #endif