On 11/18/2013 02:27 PM, Ken Steele wrote:
> In src/decode.h:
> 
>> @@ -427,6 +427,9 @@ enum PktSrcEnum {
>>      /* header pointers */
>>      EthernetHdr *ethh;
>>  
>> +    /* Check sum for TCP, UDP or ICMP packets */
>> +    int32_t comp_csum;
> 
> I like the change l4_comp_csum, although “l4” looking to much like “14”,
> so either L4_comp_csum, or level4_comp_csum? It also makes sense to add
> a L3 checksum for IPv4, otherwise I can’t union IPv4vars with IPv6vars
> as setting the csum in IPv4 will then clobber IPv6 data.

Right. Lets make it level3_* level4_* even though it's a bit verbose.