Bug 1260

Summary:	checksum of the UDP fragmented packets are never calculated.
Product:	ns-3	Reporter:	Frederic Urbani <frederic.urbani>
Component:	internet	Assignee:	George Riley <riley>
Status:	RESOLVED FIXED
Severity:	normal	CC:	bthompson, john.abraham.in, ns-bugs
Priority:	P5
Version:	ns-3-dev
Hardware:	All
OS:	All

Description Frederic Urbani 2011-09-09 09:40:11 UTC

All in title.

My Patch:

diff -r e52db23c9c8b src/internet/model/ipv4-l3-protocol.cc
--- a/src/internet/model/ipv4-l3-protocol.cc	Thu Sep 08 18:24:47 2011 -0700
+++ b/src/internet/model/ipv4-l3-protocol.cc	Fri Sep 09 15:36:53 2011 +0200
@@ -1170,6 +1170,11 @@
 
       NS_LOG_LOGIC ("Fragment check - " << fragmentHeader.GetFragmentOffset ()  );
 
+      if (Node::ChecksumEnabled ())
+        {
+          fragmentHeader.EnableChecksum ();
+        }
+
       NS_LOG_LOGIC ("New fragment Header " << fragmentHeader);
       fragment->AddHeader (fragmentHeader);

Comment 1 Frederic Urbani 2011-09-09 09:55:47 UTC

or perhaps :

diff -r e52db23c9c8b src/internet/model/ipv4-l3-protocol.cc
--- a/src/internet/model/ipv4-l3-protocol.cc	Thu Sep 08 18:24:47 2011 -0700
+++ b/src/internet/model/ipv4-l3-protocol.cc	Fri Sep 09 15:54:55 2011 +0200
@@ -1170,6 +1170,11 @@
 
       NS_LOG_LOGIC ("Fragment check - " << fragmentHeader.GetFragmentOffset ()  );
 
+      if ( ipv4Header.IsChecksumOk() )
+        {
+          fragmentHeader.EnableChecksum ();
+        }
+
       NS_LOG_LOGIC ("New fragment Header " << fragmentHeader);
       fragment->AddHeader (fragmentHeader);

Comment 2 John Abraham 2011-10-18 18:20:57 UTC

*** Bug 1281 has been marked as a duplicate of this bug. ***

Comment 3 John Abraham 2011-10-18 18:22:03 UTC

changeset 7562	be2b2c3c480c