netdev - Re: WARNING (skb_gso_segment) with tg3+bridge in 2.6.27

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] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 13 Oct 2008 15:59:34 +0800
From:	Herbert Xu <herbert@...dor.apana.org.au>
To:	David Miller <davem@...emloft.net>
Cc:	berni@...kenwald.de, netdev@...r.kernel.org
Subject: Re: WARNING (skb_gso_segment) with tg3+bridge in 2.6.27

On Mon, Oct 13, 2008 at 12:42:26AM -0700, David Miller wrote:
> 
> Herbert, is there something about checksum state changing done
> by bridging netfilter that we have to be mindful of in
> skb_gso_segment() and therefore not consider "!= CHECKSUM_PARTIAL"
> a warn-worthy case?

There may well be a bug in bridging netfilter where it destroys
the partial checksum.  However, regardless of what happens there,
it's always a bug to have a non-partial checksum on GSO packets
because the underlying driver may depend on the partial checksum
to be there when computing the checksum for the segmented packets.

I'll take a look at bridging to see if it's doing anything stupid.

Actually I was looking at these skb_gso_segment warning reports
yesterday and I think the problem is more likely to be in the
TCP stack as 90% of the traces I've seen come straight out of
there.

There is a known issue with IPv6 packets where the driver might
turn on TSO6 without IPv6 checksum offload.  However, most of
the traces were IPv4 so there must be something else going on
as well.

Cheers,
-- 
Visit Openswan at http://www.openswan.org/
Email: Herbert Xu ~{PmV>HI~} <herbert@...dor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
--
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