All of lore.kernel.org
 help / color / mirror / Atom feed
From: Victor Julien <lists@inliniac.net>
To: Eric Leblond <eric@inl.fr>
Cc: netfilter-devel@vger.kernel.org
Subject: Re: libnetfilter_queue question
Date: Fri, 21 May 2010 18:02:56 +0200	[thread overview]
Message-ID: <4BF6AEB0.1040303@inliniac.net> (raw)
In-Reply-To: <1274299848.21477.6.camel@ice-age>

Eric Leblond wrote:
> Hi Victor,
> 
> Le mercredi 19 mai 2010 à 12:15 +0200, Victor Julien a écrit :
>> Hi all,
>>
>> I'm using libnetfilter_queue for inline mode in the Suricata IDS/IPS
>> (www.openinfosecfoundation.org). I'm using a callback that makes the
>> packet(s) available to the detection engine. In some special cases the
>> call back could fail (only malloc failure atm).
>>
>> I was wondering what the proper response would be to such an event. I'm
>> assuming nfq_handle_packet() would return an (non zero) error code in
>> that case.
>>
>> Should I verdict the packet? (drop to be safe)
> 
> Yes, clearly ! If you don't do this the packet will get stuck inside the
> kernel and nothing will released it (and free associated structures).
> 
> The only other mean to free queued packet is to unregister from the NF
> queue.

Thanks Eric, I will implement the verdict in case of error.

Cheers,
Victor

-- 
---------------------------------------------
Victor Julien
http://www.inliniac.net/
PGP: http://www.inliniac.net/victorjulien.asc
---------------------------------------------

--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2010-05-21 16:03 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-05-19 10:15 libnetfilter_queue question Victor Julien
2010-05-19 20:10 ` Eric Leblond
2010-05-21 16:02   ` Victor Julien [this message]
2010-05-27 13:25     ` Victor Julien
  -- strict thread matches above, loose matches on Subject: below --
2011-05-04  6:14 nowhere
2011-05-04 18:13 ` Alessandro Vesely
2011-05-04 18:32   ` Nikolay S.
2011-05-05  9:12     ` Alessandro Vesely
2011-05-05  9:24       ` nowhere

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=4BF6AEB0.1040303@inliniac.net \
    --to=lists@inliniac.net \
    --cc=eric@inl.fr \
    --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.