From 392811a5c468960fa9c178f8de31560224ee7069 Mon Sep 17 00:00:00 2001 From: Jo-Philipp Wich Date: Wed, 7 Jun 2017 15:17:42 +0200 Subject: [PATCH] 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 --- ubus.c | 25 ++++++++++++++++--------- ubus.h | 2 +- 2 files changed, 17 insertions(+), 10 deletions(-) diff --git a/ubus.c b/ubus.c index 5bb4f5d..7029d63 100644 --- a/ubus.c +++ b/ubus.c @@ -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 diff --git a/ubus.h b/ubus.h index 1cda487..263f827 100644 --- a/ubus.h +++ b/ubus.h @@ -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); -- 2.30.2