appneta / tcpreplay

Pcap editing and replay tools for *NIX and Windows - Users please download source from
http://tcpreplay.appneta.com/wiki/installation.html#downloads
1.17k stars 268 forks source link

[Bug] tcpreplay-edit recomputes IPv4 checksums unnecessarily #815

Closed green-nsk closed 3 months ago

green-nsk commented 1 year ago

Describe the bug

I think this line causes tcpreplay-edit to recompute all checksums for all IP packets unnecessarily. I noticed this because of a problem with recomputing UDP checksums, but in general I think it is 1) not allowing users to replay pcaps with errors 2) waste of time.

To Reproduce

Replay any pcap with errors in IP/UDP/TCP header checksums.

Expected behavior

I expect checksums to stay intact unless explicitly requested

Screenshots

N/A

System (please complete the following information): All

Additional context

I tried fixing it, but seemingly unrelated tests keep breaking, so I figured I'm in over my head.

fklassen commented 3 months ago

Fixed in PR #846