Bug 1426678 - Ensure SendTrack is not null before trying to dispatch DTMF event; r?bwc
MozReview-Commit-ID: CZbvPlIceXB
--- a/media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp
+++ b/media/webrtc/signaling/src/peerconnection/PeerConnectionImpl.cpp
@@ -3825,18 +3825,24 @@ PeerConnectionImpl::DTMFState::Notify(ns
}
RefPtr<PeerConnectionObserver> pco = do_QueryObjectReferent(mPCObserver);
if (!pco) {
NS_WARNING("Failed to dispatch the RTCDTMFToneChange event!");
return NS_OK; // Return is ignored anyhow
}
+ RefPtr<dom::MediaStreamTrack> sendTrack = mTransceiver->GetSendTrack();
+ if (!sendTrack) {
+ NS_WARNING("Failed to dispatch the RTCDTMFToneChange event!");
+ return NS_OK; // Return is ignored anyhow
+ }
+
JSErrorResult jrv;
- pco->OnDTMFToneChange(*mTransceiver->GetSendTrack(), eventTone, jrv);
+ pco->OnDTMFToneChange(*sendTrack, eventTone, jrv);
if (jrv.Failed()) {
NS_WARNING("Failed to dispatch the RTCDTMFToneChange event!");
}
return NS_OK;
}