diff options
author | Arturo Borrero <arturo.borrero.glez@gmail.com> | 2014-10-10 21:43:44 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2014-10-11 12:22:51 +0200 |
commit | bc22a51c662e9b7f68dbd754396e206d9a114ed0 (patch) | |
tree | d0a800e701ae77e7a4cfeb73d1ee01f522e4d1c3 /src | |
parent | c04175e392335fb22b52f234171b5042e0b8f6bd (diff) |
src: cleanup in mxml and jansson regarding set_id parsing
jansson.c: In function 'nft_jansson_expr_parse':
jansson.c:212:6: warning: pointer targets in passing argument 3 of 'nft_set_lookup_id' differ in signedness [-Wpointer-sign]
In file included from jansson.c:10:0:
./internal.h:95:5: note: expected 'uint32_t *' but argument is of type 'int *'
jansson.c:195:18: warning: unused variable 'set_cur' [-Wunused-variable]
mxml.c: In function 'nft_mxml_expr_parse':
mxml.c:97:6: warning: pointer targets in passing argument 3 of 'nft_set_lookup_id' differ in signedness [-Wpointer-sign]
In file included from mxml.c:12:0:
internal.h:95:5: note: expected 'uint32_t *' but argument is of type 'int *'
mxml.c:68:18: warning: unused variable 'set_cur' [-Wunused-variable]
Spotted with: gcc (Debian 4.7.2-5) 4.7.2
Signed-off-by: Arturo Borrero Gonzalez <arturo.borrero.glez@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src')
-rw-r--r-- | src/jansson.c | 4 | ||||
-rw-r--r-- | src/mxml.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/src/jansson.c b/src/jansson.c index 728de12..4c7968a 100644 --- a/src/jansson.c +++ b/src/jansson.c @@ -192,8 +192,8 @@ struct nft_rule_expr *nft_jansson_expr_parse(json_t *root, { struct nft_rule_expr *e; const char *type; - struct nft_set *set_cur = NULL; - int ret, set_id; + uint32_t set_id; + int ret; type = nft_jansson_parse_str(root, "type", err); if (type == NULL) @@ -65,8 +65,8 @@ struct nft_rule_expr *nft_mxml_expr_parse(mxml_node_t *node, struct nft_rule_expr *e; const char *expr_name; char *xml_text; - struct nft_set *set_cur = NULL; - int ret, set_id; + uint32_t set_id; + int ret; expr_name = mxmlElementGetAttr(node, "type"); if (expr_name == NULL) { |