Bug 2775

Summary:	UdpClient packet size attribute is too restrictive
Product:	ns-3	Reporter:	Tom Henderson <tomh>
Component:	applications	Assignee:	George Riley <riley>
Status:	RESOLVED FIXED
Severity:	normal	CC:	ns-bugs, tommaso.pecorella
Priority:	P3
Version:	unspecified
Hardware:	All
OS:	All

Description Tom Henderson 2017-08-16 18:54:34 UTC

Comment 1 Tom Henderson 2017-08-16 18:56:16 UTC

The attribute UdpClient::PacketSize limits to 1500 bytes, which is too limiting (MTUs may be larger, or fragmentation may be tolerated).  I propose to raise the limit to the value of 65507 bytes.

diff -r f010860857bd src/applications/model/udp-client.cc
--- a/src/applications/model/udp-client.cc	Wed Aug 16 19:13:57 2017 +0300
+++ b/src/applications/model/udp-client.cc	Wed Aug 16 15:53:04 2017 -0700
@@ -68,7 +68,7 @@
                    "Size of packets generated. The minimum packet size is 12 bytes which is the size of the header carrying the sequence number and the time stamp.",
                    UintegerValue (1024),
                    MakeUintegerAccessor (&UdpClient::m_size),
-                   MakeUintegerChecker<uint32_t> (12,1500))
+                   MakeUintegerChecker<uint32_t> (12,65507))
   ;
   return tid;
 }

Comment 2 Tommaso Pecorella 2017-08-20 12:06:31 UTC

+1

Comment 3 Tom Henderson 2018-05-30 00:16:25 UTC

pushed in changeset 13606:585dab8f400f