From: Pablo Neira Ayuso <pablo@netfilter.org>
To: Florian Westphal <fw@strlen.de>
Cc: "Anders K. Pedersen | Cohaesio" <akp@cohaesio.com>,
"netfilter-devel@vger.kernel.org"
<netfilter-devel@vger.kernel.org>
Subject: Re: nftables rules not matching after upgrading from 0.7 to 0.8
Date: Thu, 26 Oct 2017 01:47:46 +0200 [thread overview]
Message-ID: <20171025234746.GA4172@salvia> (raw)
In-Reply-To: <20171025224536.GK19457@breakpoint.cc>
On Thu, Oct 26, 2017 at 12:45:36AM +0200, Florian Westphal wrote:
> Anders K. Pedersen | Cohaesio <akp@cohaesio.com> wrote:
> > On ons, 2017-10-25 at 20:20 +0200, Anders K. Pedersen | Cohaesio wrote:
> > > On ons, 2017-10-25 at 19:57 +0200, Florian Westphal wrote:
> > > > Anders K. Pedersen | Cohaesio <akp@cohaesio.com> wrote:
[...]
> > > As mentioned, it's not consistent. A rule that has worked fine could
> > > suddenly stop working without any rule set changes for days. Some
> > > times
> > > it has helped to just reload the exact same rule set. Other times
> > > changing
> > >
> > > tcp dport { imap2, imaps } flow table imap \
> > > { ip saddr & 255.255.255.240 \
> > > timeout 5m limit rate 10/minute } \
> > > counter accept
> > >
> > > to
> > >
> > > tcp dport { imap2, imaps } flow table imap \
> > > counter accept
> > >
> > > or
> > >
> > > tcp dport { domain, http, https, 8080, 8443, 9091 } \
> > > meta iif eth1 accept
> > >
> > > to
> > >
> > > tcp dport { domain, http, https, 8080, 8443, 9091 } \
> > > meta iif eth1 counter accept
> > >
> > > has resolved it, but it feels like it wasn't really due to the
> > > changes,
> > > but more random luck.
>
> note that sets are broken with 16bit elements at the moment in 4.13, see
>
> https://patchwork.ozlabs.org/patch/821080/
> or
> https://patchwork.ozlabs.org/patch/830236/
JFYI, the fix for sets is now in nf.git.
https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf.git/commit/
I swear this was included in the previous batch, but it was not. Sorry
about this. Will send this upstream asap.
next prev parent reply other threads:[~2017-10-25 23:47 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-25 16:31 nftables rules not matching after upgrading from 0.7 to 0.8 Anders K. Pedersen | Cohaesio
2017-10-25 17:57 ` Florian Westphal
2017-10-25 18:20 ` Anders K. Pedersen | Cohaesio
2017-10-25 22:22 ` Anders K. Pedersen | Cohaesio
2017-10-25 22:45 ` Florian Westphal
2017-10-25 23:44 ` Pablo Neira Ayuso
2017-10-26 7:00 ` Anders K. Pedersen | Cohaesio
2017-10-25 23:47 ` Pablo Neira Ayuso [this message]
2017-10-26 6:47 ` Anders K. Pedersen | Cohaesio
2017-11-06 16:49 ` Anders K. Pedersen | Cohaesio
2017-11-07 0:58 ` Pablo Neira Ayuso
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171025234746.GA4172@salvia \
--to=pablo@netfilter.org \
--cc=akp@cohaesio.com \
--cc=fw@strlen.de \
--cc=netfilter-devel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.