diff options
author | Jo-Philipp Wich <jo@mein.io> | 2017-06-07 15:17:42 +0200 |
---|---|---|
committer | Jo-Philipp Wich <jo@mein.io> | 2018-02-13 15:13:40 +0100 |
commit | 392811a5c468960fa9c178f8de31560224ee7069 (patch) | |
tree | f771c8e299225fee8e366b66cdaecd8e671aa6ff | |
parent | c4309372acc7e1bef8aa230269f7da1dec790e68 (diff) | |
download | firewall3-392811a5c468960fa9c178f8de31560224ee7069.tar.gz |
ubus: let fw3_ubus_address() return the number of resolved addresses
Change fw3_ubus_address() to return the number of addresses resolved from
the given network name, this will be required to handle failed resolving
in higher layers later on.
Signed-off-by: Jo-Philipp Wich <jo@mein.io>
-rw-r--r-- | ubus.c | 25 | ||||
-rw-r--r-- | ubus.h | 2 |
2 files changed, 17 insertions, 10 deletions
@@ -103,16 +103,16 @@ parse_subnet(enum fw3_family family, struct blob_attr *dict, int rem) return addr; } -static void +static int parse_subnets(struct list_head *head, enum fw3_family family, struct blob_attr *list) { struct blob_attr *cur; struct fw3_address *addr; - int rem; + int rem, n = 0; if (!list) - return; + return 0; rem = blobmsg_data_len(list); @@ -121,8 +121,13 @@ parse_subnets(struct list_head *head, enum fw3_family family, addr = parse_subnet(family, blobmsg_data(cur), blobmsg_data_len(cur)); if (addr) + { list_add_tail(&addr->list, head); + n++; + } } + + return n; } struct fw3_device * @@ -178,7 +183,7 @@ fw3_ubus_device(const char *net) return dev; } -void +int fw3_ubus_address(struct list_head *list, const char *net) { enum { @@ -196,10 +201,10 @@ fw3_ubus_address(struct list_head *list, const char *net) }; struct blob_attr *tb[__ADDR_MAX]; struct blob_attr *cur; - int rem; + int rem, n = 0; if (!net || !interfaces) - return; + return 0; blobmsg_for_each_attr(cur, interfaces, rem) { blobmsg_parse(policy, __ADDR_MAX, tb, blobmsg_data(cur), blobmsg_len(cur)); @@ -208,10 +213,12 @@ fw3_ubus_address(struct list_head *list, const char *net) strcmp(blobmsg_data(tb[ADDR_INTERFACE]), net) != 0) continue; - parse_subnets(list, FW3_FAMILY_V4, tb[ADDR_IPV4]); - parse_subnets(list, FW3_FAMILY_V6, tb[ADDR_IPV6]); - parse_subnets(list, FW3_FAMILY_V6, tb[ADDR_IPV6_PREFIX]); + n += parse_subnets(list, FW3_FAMILY_V4, tb[ADDR_IPV4]); + n += parse_subnets(list, FW3_FAMILY_V6, tb[ADDR_IPV6]); + n += parse_subnets(list, FW3_FAMILY_V6, tb[ADDR_IPV6_PREFIX]); } + + return n; } void @@ -30,7 +30,7 @@ void fw3_ubus_disconnect(void); struct fw3_device * fw3_ubus_device(const char *net); -void fw3_ubus_address(struct list_head *list, const char *net); +int fw3_ubus_address(struct list_head *list, const char *net); void fw3_ubus_zone_devices(struct fw3_zone *zone); |