add simple device config params to interface blob attributes, and add a flag that indicates if it is present

3 files changed, 29 insertions, 13 deletions

diff --git a/config.c b/config.c
index c5a2d2c..c51e379 100644
--- a/config.c
+++ b/config.c
@@ -29,7 +29,7 @@ static struct uci_context *uci_ctx;
 static struct uci_package *uci_network;
 static struct blob_buf b;
 
-static void uci_attr_to_blob(struct blob_buf *b, const char *str,
+static bool uci_attr_to_blob(struct blob_buf *b, const char *str,
 			     const char *name, enum blobmsg_type type)
 {
 	char *err;
@@ -45,20 +45,21 @@ static void uci_attr_to_blob(struct blob_buf *b, const char *str,
 		else if (!strcmp(str, "false") || !strcmp(str, "0"))
 			intval = 0;
 		else
-			return;
+			return false;
 
 		blobmsg_add_u8(b, name, intval);
 		break;
 	case BLOBMSG_TYPE_INT32:
 		intval = strtol(str, &err, 0);
 		if (*err)
-			return;
+			return false;
 
 		blobmsg_add_u32(b, name, intval);
 		break;
 	default:
-		break;
+		return false;
 	}
+	return true;
 }
 
 static void uci_array_to_blob(struct blob_buf *b, struct uci_option *o,
@@ -87,14 +88,14 @@ static void uci_array_to_blob(struct blob_buf *b, struct uci_option *o,
 	free(str);
 }
 
-static void __uci_to_blob(struct blob_buf *b, struct uci_section *s,
-			  const struct config_param_list *p)
+static int __uci_to_blob(struct blob_buf *b, struct uci_section *s,
+			 const struct config_param_list *p)
 {
 	const struct blobmsg_policy *attr = NULL;
 	struct uci_element *e;
 	struct uci_option *o;
 	void *array;
-	int i;
+	int i, ret = 0;
 
 	uci_foreach_element(&s->options, e) {
 		for (i = 0; i < p->n_params; i++) {
@@ -115,24 +116,30 @@ static void __uci_to_blob(struct blob_buf *b, struct uci_section *s,
 			array = blobmsg_open_array(b, attr->name);
 			uci_array_to_blob(b, o, p->info[i].type);
 			blobmsg_close_array(b, array);
+			ret++;
 			continue;
 		}
 
 		if (o->type == UCI_TYPE_LIST)
 			continue;
 
-		uci_attr_to_blob(b, o->v.string, attr->name, attr->type);
+		ret += uci_attr_to_blob(b, o->v.string, attr->name, attr->type);
 	}
+
+	return ret;
 }
 
-static void uci_to_blob(struct blob_buf *b, struct uci_section *s,
-			const struct config_param_list *p)
+static int uci_to_blob(struct blob_buf *b, struct uci_section *s,
+		       const struct config_param_list *p)
 {
+	int ret = 0;
 	int i;
 
-	__uci_to_blob(b, s, p);
+	ret += __uci_to_blob(b, s, p);
 	for (i = 0; i < p->n_next; i++)
-		uci_to_blob(b, s, p->next[i]);
+		ret += uci_to_blob(b, s, p->next[i]);
+
+	return ret;
 }
 
 static int
@@ -162,15 +169,19 @@ config_parse_interface(struct uci_section *s, bool alias)
 	const char *type = NULL;
 	struct blob_attr *config;
 	struct device *dev;
+	bool bridge = false;
 
 	blob_buf_init(&b, 0);
 
 	if (!alias)
 		type = uci_lookup_option_string(uci_ctx, s, "type");
-	if (type && !strcmp(type, "bridge"))
+	if (type && !strcmp(type, "bridge")) {
 		if (config_parse_bridge_interface(s))
 			return;
 
+		bridge = true;
+	}
+
 	uci_to_blob(&b, s, &interface_attr_list);
 	iface = calloc(1, sizeof(*iface));
 	if (!iface)
@@ -181,6 +192,9 @@ config_parse_interface(struct uci_section *s, bool alias)
 	if (iface->proto_handler && iface->proto_handler->config_params)
 		uci_to_blob(&b, s, iface->proto_handler->config_params);
 
+	if (!bridge && uci_to_blob(&b, s, simple_device_type.config_params))
+		iface->device_config = true;
+
 	config = malloc(blob_pad_len(b.head));
 	if (!config)
 		goto error;
diff --git a/interface.c b/interface.c
index 036df6c..4a42ad9 100644
--- a/interface.c
+++ b/interface.c
@@ -696,6 +696,7 @@ interface_change_config(struct interface *if_old, struct interface *if_new)
 	if (!if_old->config_autostart && if_new->config_autostart)
 		if_old->autostart = true;
 
+	if_old->device_config = if_new->device_config;
 	if_old->config_autostart = if_new->config_autostart;
 	if_old->ifname = if_new->ifname;
 	if_old->parent_ifname = if_new->parent_ifname;
diff --git a/interface.h b/interface.h
index 6724bb9..efd3560 100644
--- a/interface.h
+++ b/interface.h
@@ -86,6 +86,7 @@ struct interface {
 	bool available;
 	bool autostart;
 	bool config_autostart;
+	bool device_config;
 
 	time_t start_time;
 	enum interface_state state;