[Ns-bugs] [Bug 1201] New: UDP sockets need to set connected state before notifying the ConnectionSucceeded callback

[code@nsnam.ece.gatech.edu]

https://www.nsnam.org/bugzilla/show_bug.cgi?id=1201

           Summary: UDP sockets need to set connected state before
                    notifying the ConnectionSucceeded callback
           Product: ns-3
           Version: pre-release
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: normal
          Priority: P5
         Component: internet
        AssignedTo: riley at ece.gatech.edu
        ReportedBy: tomh at tomh.org
                CC: ns-bugs at isi.edu
   Estimated Hours: 0.0


Steve Marotta suggested this fix on the ns-users list here:
http://groups.google.com/group/ns-3-users/browse_thread/thread/456d72d7c38ae2a0

diff -r b5c7eb57d8d6 src/internet/model/udp-socket-impl.cc
--- a/src/internet/model/udp-socket-impl.cc    Fri Jun 24 16:08:55 2011 +0100
+++ b/src/internet/model/udp-socket-impl.cc    Mon Jun 27 23:10:36 2011 -0700
@@ -231,8 +231,8 @@
   InetSocketAddress transport = InetSocketAddress::ConvertFrom (address);
   m_defaultAddress = transport.GetIpv4 ();
   m_defaultPort = transport.GetPort ();
+  m_connected = true;
   NotifyConnectionSucceeded ();
-  m_connected = true;

   return 0;
 }

-- 
Configure bugmail: https://www.nsnam.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.