fixup! [M108 Migration][MM] Handle buffering and playback 90/287890/5
authorKoyyani Maheswari <m.koyyani@samsung.com>
Tue, 7 Feb 2023 09:34:55 +0000 (15:04 +0530)
committerBot Blink <blinkbot@samsung.com>
Fri, 10 Feb 2023 12:53:43 +0000 (12:53 +0000)
This commit fixes crash when playing audio element.
There is a case that video demuxer stream is used even if it doesn't exist.
So prevent access to invalid demuxer stream.

Reference: https://review.tizen.org/gerrit/280782/

Change-Id: I737793d799f657f4553ca4d3ffe2ea5fc2f6ed7c
Signed-off-by: Koyyani Maheswari <m.koyyani@samsung.com>
tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc

index baab703..077a93d 100644 (file)
@@ -756,6 +756,10 @@ esplusplayer_submit_status MediaPlayerESPlusPlayer::SubmitEsPacket(
 }
 
 void MediaPlayerESPlusPlayer::UpdateBufferedDtsDifference() {
+  // Ignore unless audio and video streams are valid.
+  if (!IsValid(DemuxerStream::AUDIO) || !IsValid(DemuxerStream::VIDEO))
+    return;
+
   const auto& audio_ts = last_frames_.get<DemuxerStream::AUDIO>().first;
   const auto& video_ts = last_frames_.get<DemuxerStream::VIDEO>().first;