From: Venugopal S M <sm.venugopal@samsung.com>
Date: Thu, 2 Jun 2022 17:27:51 +0000 (+0530)
Subject: [M108 Migration][MM] Ignore buffers if callback is null
X-Git-Tag: submit/tizen/20230227.160252~66
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=23a294e50d4c4fac7f5a288c6778caed73b71248;p=platform%2Fframework%2Fweb%2Fchromium-efl.git

[M108 Migration][MM] Ignore buffers if callback is null

In "video_onended_event" for no reason, buffers are sent after EOS is
received. The read-calback is null and would crash.

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

Change-Id: Icccfa6c6e35000c7993139077ade2817232051b5
Signed-off-by: Venugopal S M <sm.venugopal@samsung.com>
---

diff --git a/media/mojo/services/mojo_demuxer_stream_adapter.cc b/media/mojo/services/mojo_demuxer_stream_adapter.cc
index dcd0d90..0dcbb81 100644
--- a/media/mojo/services/mojo_demuxer_stream_adapter.cc
+++ b/media/mojo/services/mojo_demuxer_stream_adapter.cc
@@ -95,6 +95,14 @@ void MojoDemuxerStreamAdapter::OnBufferReady(
   DCHECK(read_cb_);
   DCHECK_NE(type_, UNKNOWN);
 
+#if defined(TIZEN_MULTIMEDIA)
+  if (!read_cb_) {
+    LOG(INFO) << __func__ << " (" << DemuxerStream::GetTypeName(type_)
+              << ") ignoring as cb is null!!";
+    return;
+  }
+#endif
+
   if (status == kConfigChanged) {
     UpdateConfig(std::move(audio_config), std::move(video_config));
     std::move(read_cb_).Run(kConfigChanged, nullptr);