All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stig Thormodsrud <stig@vyatta.com>
To: Patrick McHardy <kaber@trash.net>
Cc: Andrew Watts <akwatts@ymail.com>, netfilter-devel@vger.kernel.org
Subject: Re: NFQUEUE verdicts - adding non-termination
Date: Fri, 12 Nov 2010 11:54:02 -0800	[thread overview]
Message-ID: <4CDD9B5A.4050800@vyatta.com> (raw)
In-Reply-To: <4CDBCA8C.2000801@trash.net>

On 11/11/2010 02:50 AM, Patrick McHardy wrote:
> On 11.11.2010 10:01, Andrew Watts wrote:
>> Hi.
>>
>> The NF_CONTINUE verdict that Darryl Miles brings up in his 11/4 post is very interesting.
>>
>> NF_CONTINUE would provide the NFQUEUE target the added flexibility of, say, partial handling in userspace. A queue-handler could have a set of criteria that, when satisfied, would result in an immediate drop or accept. One could then leave the rest of the packets to find their fate in the chains/rules left to traverse.
>>
>> I would be interested in helping to add this verdict if someone will take the lead (assuming a patch hasn't already been written - has it?).
> 
> There's no difference between returning NF_ACCEPT or a new NF_CONTINUE.
> Queueing happens outside of the ruleset context, so in either case the
> packet would continue through the network stack directly, not after
> the NFQUEUE rule.

I've been interested in this thread since I recently converted our
version of snort inline to use NFQUEUE instead of QUEUE.  One of driving
factors was that I needed to have multiple QUEUE targets.  While I was
doing the work I was hoping there was a verdict like NF_RETURN, so that
if it passed the snort inspection then I could have another NFQUEUE
target that would do https domain filtering.  From reading this thread I
now understand why I can just continue, but I wondering what is that the
recommended approach for this type of use case?  As it is now, I think I
would need to have the 2nd NFQUEUE target on a different hook.

stig

  parent reply	other threads:[~2010-11-12 19:54 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-11  9:01 NFQUEUE verdicts - adding non-termination Andrew Watts
2010-11-11 10:50 ` Patrick McHardy
2010-11-12 11:01   ` Andrew Watts
2010-11-12 11:11     ` Patrick McHardy
2010-11-12 11:19       ` Patrick McHardy
2010-11-12 20:51         ` Andrew Watts
2010-11-12 19:54   ` Stig Thormodsrud [this message]
2010-11-15 10:34     ` Patrick McHardy
2010-11-16 10:48       ` Andrew Watts

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=4CDD9B5A.4050800@vyatta.com \
    --to=stig@vyatta.com \
    --cc=akwatts@ymail.com \
    --cc=kaber@trash.net \
    --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.