Bug 916187. Part 3. Double checking for leaks in new test stun server code, and fixing other things as I see them. r=abr
authorByron Campen [:bwc] <docfaraday@gmail.com>
Wed, 18 Sep 2013 12:54:38 -0700
changeset 161771 f7a1c35bce2aedb82921024ad9344a17e3bb1ee8
parent 161770 ce4aadef1cc56384eee2b7c02665bb75d34ce5a1
child 161772 c301037184db3db04dbd03c4bad472e8cd29c5e2
push id3066
push userakeybl@mozilla.com
push dateMon, 09 Dec 2013 19:58:46 +0000
treeherdermozilla-beta@a31a0dce83aa [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersabr
bugs916187
milestone27.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 916187. Part 3. Double checking for leaks in new test stun server code, and fixing other things as I see them. r=abr
media/mtransport/test/ice_unittest.cpp
--- a/media/mtransport/test/ice_unittest.cpp
+++ b/media/mtransport/test/ice_unittest.cpp
@@ -186,16 +186,17 @@ class IceTestPeer : public sigslot::has_
     ASSERT_TRUE(NS_SUCCEEDED(ice_ctx_->SetTurnServers(turn_servers)));
   }
 
   void SetFakeResolver() {
     ASSERT_TRUE(NS_SUCCEEDED(dns_resolver_->Init()));
     PRNetAddr addr;
     PRStatus status = PR_StringToNetAddr(kDefaultStunServerAddress.c_str(),
                                          &addr);
+    addr.inet.port = kDefaultStunServerPort;
     ASSERT_EQ(PR_SUCCESS, status);
     fake_resolver_.SetAddr(kDefaultStunServerHostname, addr);
     ASSERT_TRUE(NS_SUCCEEDED(ice_ctx_->SetResolver(
         fake_resolver_.AllocateResolver())));
   }
 
   void SetDNSResolver() {
     ASSERT_TRUE(NS_SUCCEEDED(dns_resolver_->Init()));