[Ns-bugs] [Bug 491] It is painful to enable all checksums
code at nsnam.ece.gatech.edu
Sun Apr 26 19:22:02 PDT 2009
--- Comment #2 from Craig Dowell <craigdo at ee.washington.edu> 2009-04-26 22:22:02 EDT ---
I got bit by this again today. V4Ping creates an icmp header outside of the
Icmpv4L4Protocol where there is a calcChecksum. Checksums in V4Ping aren't
turned on when you turn on checksums in icmp.
It's time to fix this.
I don't think that adding a calChecksum to InternetStackHelper is the right
thing to do. There is no law that says that a script must ever instantiate an
internet stack helper. We need something that turns on checksums irrespective
of what level script one is using.
Like I said, I think there should be a global enable-checksums option that
trumps the individual checksum attributes and just turns on all checksums in
the system. All checksum enabling code should respect this global value.
I would like to fix this tomorrow to get emu-ping working. How about adding a
GlobalValue in the internet stack module?
Configure bugmail: http://www.nsnam.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
More information about the Ns-bugs