* [PATCH v2] nftables: Fix list of sets by family
@ 2014-03-28 11:40 Ana Rey
2014-03-28 12:31 ` Patrick McHardy
0 siblings, 1 reply; 2+ messages in thread
From: Ana Rey @ 2014-03-28 11:40 UTC (permalink / raw)
To: netfilter-devel; +Cc: Ana Rey
Fix the result of command line 'nft list sets FAMILY'. It shows the
following error message:
"Error: syntax error, unexpected end of file, expecting string"
Now, it is possible shows right this information:
$ sudo nft -nna list sets ip
set set_test {
type ipv4_address
elements = { 192.168.3.45, 192.168.3.43, 192.168.3.42, 192.168.3.4}
}
set set_test2 {
type ipv4_address
elements = { 192.168.3.43, 192.168.3.42, 192.168.3.4}
}
set set0 {
type ipv4_address
flags constant
elements = { 127.0.0.12, 12.11.11.11}
}
Signed-off-by: Ana Rey <anarey@gmail.com>
---
[Changes in v2]
* Delete all references regarding the indentation in the code.
* Delete all references regarding the indentation in the descriptions.
These changes were spotted by Patrick McHardy.
src/parser.y | 2 +-
src/rule.c | 8 ++++++--
2 files changed, 7 insertions(+), 3 deletions(-)
diff --git a/src/parser.y b/src/parser.y
index db6f493..af34857 100644
--- a/src/parser.y
+++ b/src/parser.y
@@ -719,7 +719,7 @@ list_cmd : TABLE table_spec
{
$$ = cmd_alloc(CMD_LIST, CMD_OBJ_CHAIN, &$2, &@$, NULL);
}
- | SETS table_spec
+ | SETS tables_spec
{
$$ = cmd_alloc(CMD_LIST, CMD_OBJ_SETS, &$2, &@$, NULL);
}
diff --git a/src/rule.c b/src/rule.c
index b719040..00dedf5 100644
--- a/src/rule.c
+++ b/src/rule.c
@@ -668,8 +668,12 @@ static int do_command_list(struct netlink_ctx *ctx, struct cmd *cmd)
case CMD_OBJ_SETS:
if (netlink_list_sets(ctx, &cmd->handle, &cmd->location) < 0)
return -1;
- list_for_each_entry_safe(set, nset, &ctx->list, list)
- list_move_tail(&set->list, &table->sets);
+ list_for_each_entry(set, &ctx->list, list){
+ if (netlink_get_setelems(ctx, &set->handle,
+ &cmd->location, set) < 0)
+ return -1;
+ set_print(set);
+ }
break;
case CMD_OBJ_SET:
if (netlink_get_set(ctx, &cmd->handle, &cmd->location) < 0)
--
1.9.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v2] nftables: Fix list of sets by family
2014-03-28 11:40 [PATCH v2] nftables: Fix list of sets by family Ana Rey
@ 2014-03-28 12:31 ` Patrick McHardy
0 siblings, 0 replies; 2+ messages in thread
From: Patrick McHardy @ 2014-03-28 12:31 UTC (permalink / raw)
To: Ana Rey; +Cc: netfilter-devel
On Fri, Mar 28, 2014 at 12:40:31PM +0100, Ana Rey wrote:
> Fix the result of command line 'nft list sets FAMILY'. It shows the
> following error message:
>
> "Error: syntax error, unexpected end of file, expecting string"
>
> Now, it is possible shows right this information:
>
> $ sudo nft -nna list sets ip
> set set_test {
> type ipv4_address
> elements = { 192.168.3.45, 192.168.3.43, 192.168.3.42, 192.168.3.4}
> }
> set set_test2 {
> type ipv4_address
> elements = { 192.168.3.43, 192.168.3.42, 192.168.3.4}
> }
> set set0 {
> type ipv4_address
> flags constant
> elements = { 127.0.0.12, 12.11.11.11}
> }
>
> Signed-off-by: Ana Rey <anarey@gmail.com>
Applied, thanks Ana.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-03-28 12:31 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-28 11:40 [PATCH v2] nftables: Fix list of sets by family Ana Rey
2014-03-28 12:31 ` Patrick McHardy
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.