]> git.proxmox.com Git - mirror_ubuntu-bionic-kernel.git/commit - net/netfilter/nft_dynset.c
netfilter: nft_dynset: continue to next expr if _OP_ADD succeeded
authorLiping Zhang <zlpnobody@gmail.com>
Sun, 23 Apr 2017 10:29:30 +0000 (18:29 +0800)
committerPablo Neira Ayuso <pablo@netfilter.org>
Tue, 25 Apr 2017 09:10:37 +0000 (11:10 +0200)
commit277a292835c196894ef895d5e1fd6170bb916f55
tree20f742f20a4ffc9a8e1e7a524e58f4cd2bb8d635
parentcf3cb246e277d98987aa8d62ef2730dbee2f5fa7
netfilter: nft_dynset: continue to next expr if _OP_ADD succeeded

Currently, after adding the following nft rules:
  # nft add set x target1 { type ipv4_addr \; flags timeout \;}
  # nft add rule x y set add ip daddr timeout 1d @target1 counter

the counters will always be zero despite of the elements are added
to the dynamic set "target1" or not, as we will break the nft expr
traversal unconditionally:
  # nft list ruleset
  ...
  set target1 {
      ...
      elements = { 8.8.8.8 expires 23h59m53s}
  }
  chain output {
      ...
      set add ip daddr timeout 1d @target1 counter packets 0 bytes 0
                                                           ^       ^
      ...
  }

Since we add the elements to the set successfully, we should continue
to the next expression.

Additionally, if elements are added to "flow table" successfully, we
will _always_ continue to the next expr, even if the operation is
_OP_ADD. So it's better to keep them to be consistent.

Fixes: 22fe54d5fefc ("netfilter: nf_tables: add support for dynamic set updates")
Reported-by: Robert White <rwhite@pobox.com>
Signed-off-by: Liping Zhang <zlpnobody@gmail.com>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nft_dynset.c