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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 22 Jan 2007 07:52:14 +0100 From: Jarek Poplawski <jarkao2@...pl> To: Herbert Xu <herbert@...dor.apana.org.au> Cc: Michael Tokarev <mjt@....msk.ru>, netdev@...r.kernel.org, Patrick McHardy <kaber@...sh.net>, "David S\. Miller" <davem@...emloft.net> Subject: Re: [PATCH] tcp_output: Re: rare bad TCP checksum with 2.6.19? On Sat, Jan 20, 2007 at 08:10:27AM +1100, Herbert Xu wrote: > On Fri, Jan 19, 2007 at 12:06:41PM +0100, Jarek Poplawski wrote: > > > > [PATCH][NET] tcp_output: rare bad TCP checksum with 2.6.19 > > > > The patch "Replace CHECKSUM_HW by CHECKSUM_PARTIAL/CHECKSUM_COMPLETE" > > changed to unconditional copying of ip_summed field from collapsed > > skb. This patch reverts this change. > > > > All substantial work including heavy testing and diagnosing by: > > Michael Tokarev <mjt@....msk.ru> > > > > Signed-off-by: Jarek Poplawski <jarkao2@...pl> > > Acked-by: Herbert Xu <herbert@...dor.apana.org.au> > > Thanks for catching this! I'll take the credit for adding this bug :) > > Dave, we'll need this fix for 2.6.20 as well as 2.6.19. I was so impressed by the amount of work done by Michael that I magnified his merit and forgot to mention the role of Patrick and Herbert, particularly here: > Since you're certain that this is being seen on the wire, one > possibility is that we've got a bug somewhere that's zeroing > skb->ip_summed on a packet with a partial checksum. which exactly pointed the reason. So, I apologize to them and, if there is such possibility, I would like to ask David Miller to change the description like that: --- [PATCH][NET] tcp_output: rare bad TCP checksum with 2.6.19 The patch "Replace CHECKSUM_HW by CHECKSUM_PARTIAL/CHECKSUM_COMPLETE" changed to unconditional copying of ip_summed field from collapsed skb. This patch reverts this change. The majority of substantial work including heavy testing and diagnosing by: Michael Tokarev <mjt@....msk.ru> Possible reasons pointed by: Herbert Xu and Patrick McHardy. Signed-off-by: Jarek Poplawski <jarkao2@...pl> Acked-by: Herbert Xu <herbert@...dor.apana.org.au> --- 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
Powered by blists - more mailing lists