netdev - Re: [v5 PATCH 1/3] NETFILTER added flags to __ipv6_find

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: <201201042148.11614.hans@schillstrom.com>
Date:	Wed, 4 Jan 2012 21:48:11 +0100
From:	Hans Schillstrom <hans@...illstrom.com>
To:	Pablo Neira Ayuso <pablo@...filter.org>
Cc:	Hans Schillstrom <hans.schillstrom@...csson.com>, kaber@...sh.net,
	jengelh@...ozas.de, netfilter-devel@...r.kernel.org,
	netdev@...r.kernel.org
Subject: Re: [v5 PATCH 1/3] NETFILTER added flags to __ipv6_find_hdr()


On Wednesday, January 04, 2012 18:37:41 Pablo Neira Ayuso wrote:
> On Mon, Jan 02, 2012 at 04:06:39PM +0100, Hans Schillstrom wrote:
> > Two new flags to __ipv6_find_hdr,
> > One that tells us that this is a fragemnt.
> > One that stops at AH if any i.e. treat it like a transport header.
> > i.e. make handling of ESP and AH the same.
> > 
> > Signed-off-by: Hans Schillstrom <hans.schillstrom@...csson.com>
> > ---
> >  include/linux/netfilter_ipv6/ip6_tables.h |   16 ++++++++++++++--
> >  net/ipv6/netfilter/ip6_tables.c           |   19 ++++++++++++++-----
> >  2 files changed, 28 insertions(+), 7 deletions(-)
> > 
> > diff --git a/include/linux/netfilter_ipv6/ip6_tables.h b/include/linux/netfilter_ipv6/ip6_tables.h
> > index f549adc..ee0c68e 100644
> > --- a/include/linux/netfilter_ipv6/ip6_tables.h
> > +++ b/include/linux/netfilter_ipv6/ip6_tables.h
> > @@ -288,9 +288,21 @@ extern unsigned int ip6t_do_table(struct sk_buff *skb,
> >  
> >  /* Check for an extension */
> >  extern int ip6t_ext_hdr(u8 nexthdr);
> > +enum {
> > +	IP6T_FH_FRAG,
> > +	IP6T_FH_AUTH,
> > +	IP6T_FH_F_FRAG = 1 << IP6T_FH_FRAG,
> > +	IP6T_FH_F_AUTH = 1 << IP6T_FH_AUTH,
> > +};
> >  /* find specified header and get offset to it */
> > -extern int ipv6_find_hdr(const struct sk_buff *skb, unsigned int *offset,
> > -			 int target, unsigned short *fragoff);
> > +extern int __ipv6_find_hdr(const struct sk_buff *skb, unsigned int *offset,
> > +			 int target, unsigned short *fragoff, int *fragflg);
> 
> Please, don't do this.
> 
> the convention in the kernel is to use __function for non-locked
> versions of one function.
> 
> The number of clients for this function seems small. I'll be very
> happy if you send me a patch that changes this interface and that
> propagates the changes to other clients of it.
> 
No problem, I'll fix this.

I have some minor compilation warnings with out nat to fix also in the other patch.

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