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

Reply via email to