Re: [PATCH nf-next] netfilter: ctnetlink: make more information available in DESTROY events

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jan 18, 2017 at 03:54:32PM +0100, Florian Westphal wrote:
> destroy events currently don't contain the tcp state info and no
> secmark and conntrack labels.
> 
> Quoting Victor:
>  "I was hoping to get the last TCP state in a conntrack destroy event,
>   however it seems to be unavailable."
> 
> Quoting Jarno:
>  "I have a use case where we want to log terminating connections, but
>  only if a specific label bit is set."
> 
> While at it, also include SECMARK in destroy events if one is available.

I'm fine with this.

But to remember the original problem is that netlink bandwidth is
limited, so the more we load the netlink message, the more chances we
have to hit ENOBUFS.

connlabel is optional, so you only get it if needed.

But the protoinfo thing, I would prefer we just dump the state given
this the usecase we have now.

Probably extend ->to_nlattr() to have a bool that indicates if this is
a dump?
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Netfitler Users]     [LARTC]     [Bugtraq]     [Yosemite Forum]

  Powered by Linux