diff options
author | Álvaro Neira Ayuso <alvaroneay@gmail.com> | 2013-07-31 15:20:59 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2013-07-31 16:42:53 +0200 |
commit | 03e3740d654e1e14ccfb8dbfaba605ae72067ef6 (patch) | |
tree | e9f05c32b12a61e8e44dc4daf5df08478c4f7aed | |
parent | 72ba595e37a73c539aa64d22b2dace3e5bbea3bc (diff) |
jansson: Add helper function for building the tree and use it
Add a helper function that parses and returns the jansson
tree, use it in the table parser.
Signed-off-by: Alvaro Neira Ayuso <alvaroneay@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
-rw-r--r-- | src/internal.h | 1 | ||||
-rw-r--r-- | src/jansson.c | 19 | ||||
-rw-r--r-- | src/table.c | 14 |
3 files changed, 23 insertions, 11 deletions
diff --git a/src/internal.h b/src/internal.h index a8ae431..d1c7690 100644 --- a/src/internal.h +++ b/src/internal.h @@ -45,6 +45,7 @@ int nft_jansson_value_parse_val(json_t *root, const char *tag, int type, void *out); const char *nft_jansson_value_parse_str(json_t *root, const char *tag); bool nft_jansson_node_exist(json_t *root, const char *tag); +json_t *nft_jansson_get_root(char *json, const char *tag, json_error_t *err); #endif const char *nft_family2str(uint32_t family); diff --git a/src/jansson.c b/src/jansson.c index cc68ae0..4c778d9 100644 --- a/src/jansson.c +++ b/src/jansson.c @@ -71,4 +71,23 @@ bool nft_jansson_node_exist(json_t *root, const char *tag) { return json_object_get(root, tag) != NULL; } + +json_t *nft_jansson_get_root(char *json, const char *tag, json_error_t *err) +{ + json_t *root; + + root = json_loadb(json, strlen(json), 0, err); + if (root == NULL) { + errno = EINVAL; + return NULL; + } + + root = json_object_get(root, tag); + if (root == NULL) { + errno = EINVAL; + return NULL; + } + + return root; +} #endif diff --git a/src/table.c b/src/table.c index 1f4fe76..526f3e7 100644 --- a/src/table.c +++ b/src/table.c @@ -290,17 +290,9 @@ static int nft_table_json_parse(struct nft_table *t, char *json) const char *str; int family; - root = json_loadb(json, strlen(json), 0, &error); - if (!root) { - errno = EINVAL; - goto err; - } - - root = json_object_get(root, "table"); - if (root == NULL) { - errno = EINVAL; - goto err; - } + root = nft_jansson_get_root(json, "table", &error); + if (root == NULL) + return -1; str = nft_jansson_value_parse_str(root, "name"); if (str == NULL) |