From: Felix Fietkau <nbd@openwrt.org>
Date: Tue, 24 Sep 2013 09:01:07 +0000 (+0200)
Subject: proto-shell: move script handler config parse code to handler.c
X-Git-Url: http://git.archive.openwrt.org/?p=project%2Fnetifd.git;a=commitdiff_plain;h=f88b0173d22e7f86ad8441c9a5bc76cee77f721a

proto-shell: move script handler config parse code to handler.c

Signed-off-by: Felix Fietkau <nbd@openwrt.org>
---

diff --git a/handler.c b/handler.c
index 4d0a57a..f5db438 100644
--- a/handler.c
+++ b/handler.c
@@ -117,3 +117,60 @@ void netifd_init_script_handlers(int dir_fd, script_dump_cb cb)
 		netifd_init_script_handler(g.gl_pathv[i], cb);
 	netifd_dir_pop(prev_fd);
 }
+
+char *
+netifd_handler_parse_config(struct uci_blob_param_list *config, json_object *obj)
+{
+	struct blobmsg_policy *attrs;
+	char *str_buf, *str_cur;
+	int str_len = 0;
+	int i;
+
+	config->n_params = json_object_array_length(obj);
+	attrs = calloc(1, sizeof(*attrs) * config->n_params);
+	if (!attrs)
+		return NULL;
+
+	config->params = attrs;
+	for (i = 0; i < config->n_params; i++) {
+		json_object *cur, *name, *type;
+
+		cur = json_check_type(json_object_array_get_idx(obj, i), json_type_array);
+		if (!cur)
+			goto error;
+
+		name = json_check_type(json_object_array_get_idx(cur, 0), json_type_string);
+		if (!name)
+			goto error;
+
+		type = json_check_type(json_object_array_get_idx(cur, 1), json_type_int);
+		if (!type)
+			goto error;
+
+		attrs[i].name = json_object_get_string(name);
+		attrs[i].type = json_object_get_int(type);
+		if (attrs[i].type > BLOBMSG_TYPE_LAST)
+			goto error;
+
+		str_len += strlen(attrs[i].name) + 1;
+	}
+
+	str_buf = malloc(str_len);
+	if (!str_buf)
+		goto error;
+
+	str_cur = str_buf;
+	for (i = 0; i < config->n_params; i++) {
+		const char *name = attrs[i].name;
+
+		attrs[i].name = str_cur;
+		str_cur += sprintf(str_cur, "%s", name) + 1;
+	}
+
+	return str_buf;
+
+error:
+	free(attrs);
+	config->n_params = 0;
+	return NULL;
+}
diff --git a/handler.h b/handler.h
index 02438f2..de403e1 100644
--- a/handler.h
+++ b/handler.h
@@ -18,7 +18,20 @@
 
 typedef void (*script_dump_cb)(const char *name, json_object *obj);
 
+static inline json_object *
+json_check_type(json_object *obj, json_type type)
+{
+	if (!obj)
+		return NULL;
+
+	if (json_object_get_type(obj) != type)
+		return NULL;
+
+	return obj;
+}
+
 int netifd_open_subdir(const char *name);
 void netifd_init_script_handlers(int dir_fd, script_dump_cb cb);
+char *netifd_handler_parse_config(struct uci_blob_param_list *config, json_object *obj);
 
 #endif
diff --git a/proto-shell.c b/proto-shell.c
index 6ac8dab..d383cbf 100644
--- a/proto-shell.c
+++ b/proto-shell.c
@@ -734,79 +734,10 @@ error:
 	return NULL;
 }
 
-static json_object *
-check_type(json_object *obj, json_type type)
-{
-	if (!obj)
-		return NULL;
-
-	if (json_object_get_type(obj) != type)
-		return NULL;
-
-	return obj;
-}
-
 static inline json_object *
 get_field(json_object *obj, const char *name, json_type type)
 {
-	return check_type(json_object_object_get(obj, name), type);
-}
-
-static char *
-proto_shell_parse_config(struct uci_blob_param_list *config, json_object *obj)
-{
-	struct blobmsg_policy *attrs;
-	char *str_buf, *str_cur;
-	int str_len = 0;
-	int i;
-
-	config->n_params = json_object_array_length(obj);
-	attrs = calloc(1, sizeof(*attrs) * config->n_params);
-	if (!attrs)
-		return NULL;
-
-	config->params = attrs;
-	for (i = 0; i < config->n_params; i++) {
-		json_object *cur, *name, *type;
-
-		cur = check_type(json_object_array_get_idx(obj, i), json_type_array);
-		if (!cur)
-			goto error;
-
-		name = check_type(json_object_array_get_idx(cur, 0), json_type_string);
-		if (!name)
-			goto error;
-
-		type = check_type(json_object_array_get_idx(cur, 1), json_type_int);
-		if (!type)
-			goto error;
-
-		attrs[i].name = json_object_get_string(name);
-		attrs[i].type = json_object_get_int(type);
-		if (attrs[i].type > BLOBMSG_TYPE_LAST)
-			goto error;
-
-		str_len += strlen(attrs[i].name) + 1;
-	}
-
-	str_buf = malloc(str_len);
-	if (!str_buf)
-		goto error;
-
-	str_cur = str_buf;
-	for (i = 0; i < config->n_params; i++) {
-		const char *name = attrs[i].name;
-
-		attrs[i].name = str_cur;
-		str_cur += sprintf(str_cur, "%s", name) + 1;
-	}
-
-	return str_buf;
-
-error:
-	free(attrs);
-	config->n_params = 0;
-	return NULL;
+	return json_check_type(json_object_object_get(obj, name), type);
 }
 
 static void
@@ -818,7 +749,7 @@ proto_shell_add_handler(const char *script, json_object *obj)
 	const char *name;
 	char *str;
 
-	if (!check_type(obj, json_type_object))
+	if (!json_check_type(obj, json_type_object))
 		return;
 
 	tmp = get_field(obj, "name", json_type_string);
@@ -850,7 +781,7 @@ proto_shell_add_handler(const char *script, json_object *obj)
 
 	config = get_field(obj, "config", json_type_array);
 	if (config)
-		handler->config_buf = proto_shell_parse_config(&handler->config, config);
+		handler->config_buf = netifd_handler_parse_config(&handler->config, config);
 
 	DPRINTF("Add handler for script %s: %s\n", script, proto->name);
 	add_proto_handler(proto);