From b88ab6b38b2a9c7ac864741f3d1909cd8ae0ad42 Mon Sep 17 00:00:00 2001 From: Koyyani Maheswari Date: Tue, 7 Feb 2023 15:04:55 +0530 Subject: [PATCH] fixup! [M108 Migration][MM] Handle buffering and playback 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 --- tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc b/tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc index baab703..077a93d 100644 --- a/tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc +++ b/tizen_src/chromium_impl/media/filters/media_player_esplusplayer.cc @@ -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().first; const auto& video_ts = last_frames_.get().first; -- 2.7.4