Herbert Xu wrote: > On Thu, Aug 03, 2006 at 11:32:32AM +0200, Patrick McHardy wrote: > >>The checksum is verified here because a full checksum update is done >>later in that function and we don't want to accidentally fix up >>broken checksums. > > > Sorry, I missed that. > > BTW, we should make sure that we clear CHECKSUM_COMPLETE since at the > end of the function skb->csum no longer contains the payload checksum.
nf_ip_checksum sets skb->ip_summed to CHECKSUM_UNNECESSARY in case the checksum was correct. If not the packet is dropped, so I think it should be fine. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html