Fix some Erizo crashes and client logs (#1046)

lynckia · Oct 6, 2017 · 4561131 · 4561131
1 parent 6547cc6
commit 4561131
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/erizo/src/erizo/DtlsTransport.cpp b/erizo/src/erizo/DtlsTransport.cpp
@@ -335,6 +335,9 @@ void DtlsTransport::updateIceState(IceState state, IceConnection *conn) {
 }
 
 void DtlsTransport::updateIceStateSync(IceState state, IceConnection *conn) {
+  if (!running_) {
+    return;
+  }
   ELOG_DEBUG("%s message:IceState, transportName: %s, state: %d, isBundle: %d",
              toLog(), transport_name.c_str(), state, bundle_);
   if (state == IceState::INITIAL && this->getTransportState() != TRANSPORT_STARTED) {

diff --git a/erizo/src/erizo/WebRtcConnection.cpp b/erizo/src/erizo/WebRtcConnection.cpp
@@ -166,6 +166,10 @@ bool WebRtcConnection::createOffer(bool videoEnabled, bool audioEnabled, bool bu
 bool WebRtcConnection::setRemoteSdp(const std::string &sdp) {
   ELOG_DEBUG("%s message: setting remote SDP", toLog());
 
+  if (!sending_) {
+    return true;
+  }
+
   remoteSdp_.initWithSdp(sdp, "");
 
   if (remoteSdp_.videoBandwidth != 0) {

diff --git a/erizo_controller/erizoClient/src/Socket.js b/erizo_controller/erizoClient/src/Socket.js
@@ -99,7 +99,7 @@ const Socket = (newIo) => {
 
     socket.on('connection_failed', () => {
       Logger.error('connection failed, id:', that.id);
-      emit('connection_failed');
+      emit('connection_failed', { streamId: that.id });
     });
     socket.on('error', (err) => {
       Logger.warning('socket error, id:', that.id, ', error:', err.message);