From: jiyong.min <jiyong.min@samsung.com>
Date: Tue, 18 Jul 2023 00:09:55 +0000 (+0900)
Subject: tizenwlsink: Add null checking for 'tizen_video_viewport'
X-Git-Tag: accepted/tizen/7.0/unified/20230719.021528^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=3e1d65ab7c0ae0cc95fa43cf14ad2c22edcde40f;p=platform%2Fupstream%2Fgst-plugins-tizen.git

tizenwlsink: Add null checking for 'tizen_video_viewport'

- Previously, it's null if memory allocation is failed.

[Version] 1.20.0-19
[Issue Type] Bug fix

Change-Id: I00bbf350dede8c4df8702984ba18a0ee5b72eca2
---

diff --git a/packaging/gst-plugins-tizen.spec b/packaging/gst-plugins-tizen.spec
index cca96cc..14c5518 100644
--- a/packaging/gst-plugins-tizen.spec
+++ b/packaging/gst-plugins-tizen.spec
@@ -5,7 +5,7 @@
 Name:       gst-plugins-tizen
 Version:    1.20.0
 Summary:    GStreamer tizen plugins (common)
-Release:    18
+Release:    19
 Group:      Multimedia/Framework
 Url:        http://gstreamer.freedesktop.org/
 License:    LGPL-2.1+
diff --git a/tizenwlsink/src/wlwindow.c b/tizenwlsink/src/wlwindow.c
index ac1ca68..e00c678 100644
--- a/tizenwlsink/src/wlwindow.c
+++ b/tizenwlsink/src/wlwindow.c
@@ -302,16 +302,18 @@ gst_wl_window_new_internal (GstWlDisplay * display)
       ("tizen_video_viewport(tizen_viewport)@%p = tizen_video_get_viewport(tizen_video@%p, video_surface(wl_surface)@%p)",
       window->tizen_video_viewport, display->tizen_video,
       window->video_surface);
-  window->tizen_video_dest_mode =
-      tizen_viewport_get_destination_mode (window->tizen_video_viewport);
-  GST_WARNING
-      ("tizen_video_dest_mode(tizen_destination_mode)@%p = tizen_viewport_get_destination_mode (tizen_video_viewport@%p)",
-      window->tizen_video_dest_mode, window->tizen_video_viewport);
-  /* video surface always follow area surface by below code. */
-  GST_WARNING
-      ("tizen_viewport_follow_parent_transform(tizen_video_viewport@%p)",
-      window->tizen_video_viewport);
-  tizen_viewport_follow_parent_transform (window->tizen_video_viewport);
+  if (window->tizen_video_viewport) {
+    window->tizen_video_dest_mode =
+        tizen_viewport_get_destination_mode (window->tizen_video_viewport);
+    GST_WARNING
+        ("tizen_video_dest_mode(tizen_destination_mode)@%p = tizen_viewport_get_destination_mode (tizen_video_viewport@%p)",
+        window->tizen_video_dest_mode, window->tizen_video_viewport);
+    /* video surface always follow area surface by below code. */
+    GST_WARNING
+        ("tizen_viewport_follow_parent_transform(tizen_video_viewport@%p)",
+        window->tizen_video_viewport);
+    tizen_viewport_follow_parent_transform (window->tizen_video_viewport);
+  }
 
 #else
   window->area_viewport = wl_scaler_get_viewport (display->scaler,