netdev - Re: [Bugme-new] [Bug 8057] New: slab corruption running ip6sic

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: Windows password security audit tool. GUI, reports in PDF.

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

Date:	Wed, 25 Apr 2007 14:05:33 +0200
From:	Jarek Poplawski <jarkao2@...pl>
To:	Eric Sesterhenn / Snakebyte <snakebyte@....de>
Cc:	Herbert Xu <herbert@...dor.apana.org.au>, davem@...emloft.net,
	akpm@...ux-foundation.org, netdev@...r.kernel.org,
	bugme-daemon@...zilla.kernel.org
Subject: Re: [Bugme-new] [Bug 8057] New: slab corruption running ip6sic

On Wed, Apr 25, 2007 at 10:27:59AM +0200, Eric Sesterhenn / Snakebyte wrote:
> * Herbert Xu (herbert@...dor.apana.org.au) wrote:
> > Jarek Poplawski <jarkao2@...pl> wrote:
> > >
> > > My proposal is: maybe Eric could change this in
> > > xfrm6_tunnel_rcv() from xfrm6_tunnel.c e.g. like this:
> > > 
> > > return xfrm6_rcv_spi(skb, spi) > 0 ? : 0;
> > > 
> > > and, if no errors in testing, he could resubmit this patch? 
> > 
> > I agree, this is the right fix.
> 
> 
> The fix proposed by Jarek indeed fixes the problem, tested on two boxes,
> with an -rc5 kernel and a yesterdays git
> 
> Acked-by: Eric Sesterhenn <snakebyte@....de>
> 
> --- linux-2.6/net/ipv6/xfrm6_tunnel.c.orig	2007-04-25 00:22:30.000000000 +0200
> +++ linux-2.6/net/ipv6/xfrm6_tunnel.c	2007-04-25 00:22:45.000000000 +0200
> @@ -261,7 +261,7 @@ static int xfrm6_tunnel_rcv(struct sk_bu
>  	__be32 spi;
>  
>  	spi = xfrm6_tunnel_spi_lookup((xfrm_address_t *)&iph->saddr);
> -	return xfrm6_rcv_spi(skb, spi);
> +	return xfrm6_rcv_spi(skb, spi) > 0 ? : 0;
>  }
>  
>  static int xfrm6_tunnel_err(struct sk_buff *skb, struct inet6_skb_parm *opt,

I think the main idea of fixing the problem plus testing is Eric's only
credit and I've only proposed some change in placement of cosmetic value.

So, Eric, considering all massive work you've done with rather feeble
support, please, fix the comment and sign this patch (at least I'm not
going to).

I also please Andrew to change the assignement of this patch in -mm.

Thanks & regards,
Jarek P.
-
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