David McCullough checked in this changeset on 2010 November 5. It included these two lines: + irs->ipp->check = 0; + irs->ipp->check = ip_fast_csum((char *) irs->ipp, irs->ipp->ihl); The first line seems pointless. Was it the intention that two different fields be assigned here?