proto-shell: allow passing netmask in ip address format
authorFelix Fietkau <nbd@openwrt.org>
Thu, 13 Oct 2011 22:30:54 +0000 (00:30 +0200)
committerFelix Fietkau <nbd@openwrt.org>
Thu, 13 Oct 2011 22:30:54 +0000 (00:30 +0200)
proto-shell.c
proto-static.c
proto.c
proto.h

index f5a5fb14b88e1c861ec58b72b52b9a773b6bc02e..c90f65c7004a7cf57fb17b45339d866d07543acc 100644 (file)
@@ -236,7 +236,7 @@ enum {
 
 static const struct blobmsg_policy route_attr[__ROUTE_LAST] = {
        [ROUTE_TARGET] = { .name = "target", .type = BLOBMSG_TYPE_STRING },
-       [ROUTE_MASK] = { .name = "mask", .type = BLOBMSG_TYPE_INT32 },
+       [ROUTE_MASK] = { .name = "mask", .type = BLOBMSG_TYPE_STRING },
        [ROUTE_GATEWAY] = { .name = "gateway", .type = BLOBMSG_TYPE_STRING },
        [ROUTE_DEVICE] = { .name = "device", .type = BLOBMSG_TYPE_STRING },
 };
@@ -352,7 +352,7 @@ proto_shell_update_link(struct proto_shell_state *state, struct blob_attr **tb)
        }
 
        if (!tb[NOTIFY_IFNAME]) {
-               if (!state->iface->main_dev.dev)
+               if (!state->proto.iface->main_dev.dev)
                        return UBUS_STATUS_INVALID_ARGUMENT;
        } else if (!state->l3_dev.dev) {
                device_add_user(&state->l3_dev,
index f5d676a7bf1e05959edb71b0e135871213e151cc..b8f341238a0144754a9c31457278491394d82999 100644 (file)
@@ -103,20 +103,18 @@ static int
 proto_apply_static_settings(struct interface *iface, struct blob_attr *attr)
 {
        struct blob_attr *tb[__OPT_MAX];
-       struct in_addr ina;
        const char *error;
-       int netmask = 32;
+       unsigned int netmask = 32;
        int n_v4 = 0, n_v6 = 0;
 
        blobmsg_parse(static_attrs, __OPT_MAX, tb, blob_data(attr), blob_len(attr));
 
        if (tb[OPT_NETMASK]) {
-               if (!inet_aton(blobmsg_data(tb[OPT_NETMASK]), &ina)) {
+               netmask = parse_netmask_string(blobmsg_data(tb[OPT_NETMASK]), false);
+               if (netmask > 32) {
                        error = "INVALID_NETMASK";
                        goto error;
                }
-
-               netmask = 32 - fls(~(ntohl(ina.s_addr)));
        }
 
        if (tb[OPT_IPADDR])
diff --git a/proto.c b/proto.c
index 786d6005f9e20d3600223e99050f6888156d16c4..bd3b09ae7d2748f665ea1138d20e29edd0aab4ae 100644 (file)
--- a/proto.c
+++ b/proto.c
 
 static struct avl_tree handlers;
 
+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)
+split_netmask(char *str, unsigned int *netmask, bool v6)
 {
-       char *delim, *err = NULL;
+       char *delim = strchr(str, '/');
 
-       delim = strchr(str, '/');
        if (delim) {
                *(delim++) = 0;
 
-               *netmask = strtoul(delim, &err, 10);
-               if (err && *err)
-                       return false;
+               *netmask = parse_netmask_string(delim, v6);
        }
        return true;
 }
@@ -34,7 +58,7 @@ 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))
+       if (!split_netmask(astr, netmask, af == AF_INET6))
                return 0;
 
        if (af == AF_INET6) {
diff --git a/proto.h b/proto.h
index 32f4ffcb68e6e464c538bdc653387b31e308f1c3..57426bd70accdcc8f7542783bb5f3c726bc6f3a8 100644 (file)
--- a/proto.h
+++ b/proto.h
@@ -53,5 +53,6 @@ void proto_attach_interface(struct interface *iface, const char *proto_name);
 int interface_proto_event(struct interface_proto_state *proto,
                          enum interface_proto_cmd cmd, bool force);
 struct device_addr *proto_parse_ip_addr_string(const char *str, bool v6, int mask);
+unsigned int parse_netmask_string(const char *str, bool v6);
 
 #endif