Phil Sutter <[email protected]> wrote:
> On Fri, Feb 08, 2019 at 03:24:24PM +0100, Florian Westphal wrote:
> > Phil Sutter <[email protected]> wrote:
> > > This will be used later to identify ebtables user-defined chain policy
> > > rules.
> > > 
> > > Signed-off-by: Phil Sutter <[email protected]>
> > > ---
> > >  iptables/nft.c | 3 +++
> > >  1 file changed, 3 insertions(+)
> > > 
> > > diff --git a/iptables/nft.c b/iptables/nft.c
> > > index 8d0d10177f5ed..4010ccd51d5aa 100644
> > > --- a/iptables/nft.c
> > > +++ b/iptables/nft.c
> > > @@ -1115,6 +1115,7 @@ int add_counters(struct nftnl_rule *r, uint64_t 
> > > packets, uint64_t bytes)
> > >  
> > >  enum udata_type {
> > >   UDATA_TYPE_COMMENT,
> > > + UDATA_TYPE_EBTABLES_POLICY,
> > 
> > Pablo, do you see any problems with this?
> > I'm a bit concerned native nft could clash with this.
> 
> Short-term best would be to extend enum udata_type in nftables'
> include/rule.h by this type to avoid accidental reuse. Mid-term we
> should move all the udata_type definitions into libnftnl along with
> getters/setters to centralize management.

I'm fine with that; Pablo, just let me know and I will push Phils
patches.

Reply via email to