Re: net: Fix skb csum races when peeking

2015-07-15 Thread David Miller
From: Herbert Xu Date: Mon, 13 Jul 2015 20:01:42 +0800 > When we calculate the checksum on the recv path, we store the > result in the skb as an optimisation in case we need the checksum > again down the line. > > This is in fact bogus for the MSG_PEEK case as this is done without > any locking.

Re: net: Fix skb csum races when peeking

2015-07-13 Thread Eric Dumazet
On Mon, 2015-07-13 at 20:01 +0800, Herbert Xu wrote: > ---8<--- > When we calculate the checksum on the recv path, we store the > result in the skb as an optimisation in case we need the checksum > again down the line. > > This is in fact bogus for the MSG_PEEK case as this is done without > any

Re: net: Fix skb csum races when peeking

2015-07-13 Thread Herbert Xu
On Mon, Jul 13, 2015 at 08:01:42PM +0800, Herbert Xu wrote: > > PS we seem to no longer use the hardware checksum in case of > CHECKSUM_COMPLETE, I wonder why that is? Nevermind, it's still there. I was just looking in the wrong place. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au

net: Fix skb csum races when peeking

2015-07-13 Thread Herbert Xu
On Mon, Jul 13, 2015 at 04:31:00PM +0800, Herbert Xu wrote: > On Mon, Jul 13, 2015 at 10:28:19AM +0200, Eric Dumazet wrote: > > > > Except that udp checksum are checked outside of spinlock protection. > > Good point. I wonder when this got broken. I'll do some digging. OK looks like I can claim