From f88b0173d22e7f86ad8441c9a5bc76cee77f721a Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Tue, 24 Sep 2013 11:01:07 +0200 Subject: proto-shell: move script handler config parse code to handler.c Signed-off-by: Felix Fietkau --- proto-shell.c | 75 +++-------------------------------------------------------- 1 file changed, 3 insertions(+), 72 deletions(-) (limited to 'proto-shell.c') 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); -- cgit v1.2.1