commit c5173de511316987c873a3f0ce348db309f7b9ea
author Alexander Afanasyev <alexander.afanasyev@ucla.edu> Thu Dec 18 10:51:35 2014 -0800
committer Alexander Afanasyev <alexander.afanasyev@ucla.edu> Thu Dec 18 13:12:42 2014 -0800
tree a72157c96716f8f47a358b4dd6d0dc3a38d2a7ed
parent b9d439d202c652b15af4a2182c372a4b365e2b69

face: Don't crash when attempting to create UdpFace over a half-working connection

Change-Id: Ib32128074f8dfe834fde033517325f994d648767
Refs: #2311

daemon/face/udp-channel.cpp

diff --git a/daemon/face/udp-channel.cpp b/daemon/face/udp-channel.cpp
index c4bb0e8..2ab4b77 100644
--- a/daemon/face/udp-channel.cpp
+++ b/daemon/face/udp-channel.cpp
@@ -229,7 +229,13 @@
     clientSocket->open(m_localEndpoint.protocol());
     clientSocket->set_option(ip::udp::socket::reuse_address(true));
     clientSocket->bind(m_localEndpoint);
-    clientSocket->connect(m_newRemoteEndpoint);
+    boost::system::error_code ec;
+    clientSocket->connect(m_newRemoteEndpoint, ec);
+    if (ec) {
+      NFD_LOG_WARN("Error while creating on-demand UDP face from " << m_newRemoteEndpoint << ": "
+                   << boost::system::system_error(ec).what());
+      return;
+    }
 
     face = createFace(clientSocket,
                       onFaceCreatedNewPeerCallback,