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/mxml.c | |
| 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/mxml.c')
| -rw-r--r-- | src/mxml.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -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) { |