Merge pull request #1565 from Feodor2/master

Prevent null deref crash in WebrtcVideoConduit DTOR

1 files changed, 7 insertions, 0 deletions

diff --git a/media/webrtc/signaling/src/media-conduit/VideoConduit.cpp b/media/webrtc/signaling/src/media-conduit/VideoConduit.cpp
index b406fded58..95f599be46 100755
--- a/media/webrtc/signaling/src/media-conduit/VideoConduit.cpp
+++ b/media/webrtc/signaling/src/media-conduit/VideoConduit.cpp
@@ -521,6 +521,8 @@ WebrtcVideoConduit::DeleteStreams()
     mVideoCodecStat->EndOfCallStats();
   }
   mVideoCodecStat = nullptr;
+  //This does Release AudioConduit before mPtrViEBase set nullptr.
+  SyncTo(nullptr);
   // We can't delete the VideoEngine until all these are released!
   // And we can't use a Scoped ptr, since the order is arbitrary
   mPtrViEBase = nullptr;
@@ -543,6 +545,11 @@ WebrtcVideoConduit::SyncTo(WebrtcAudioConduit *aConduit)
 {
   CSFLogDebug(logTag, "%s Synced to %p", __FUNCTION__, aConduit);
 
+  if (!mPtrViEBase) {
+    // ViEBase has already been released; we no longer have a conduit.
+    mSyncedTo = nullptr;
+    return;
+  }
   // SyncTo(value) syncs to the AudioConduit, and if already synced replaces
   // the current sync target.  SyncTo(nullptr) cancels any existing sync and
   // releases the strong ref to AudioConduit.