From: Eunhye Choi <eunhae1.choi@samsung.com>
Date: Thu, 30 May 2024 11:13:29 +0000 (+0900)
Subject: base:subparse: fix invalid mem access issue
X-Git-Tag: accepted/tizen/7.0/unified/20240603.162303^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=09df9d16a5cba43358e89cf1ca219f805f3a41e0;p=platform%2Fupstream%2Fgstreamer.git

base:subparse: fix invalid mem access issue

- alloc memory include extra to avoid invalid memory access
  during looping based on attr[idx]

Change-Id: I52e164f26748483331361849a56c8a0bedd8f312
---

diff --git a/packaging/gstreamer.spec b/packaging/gstreamer.spec
index 934467ade8..f275624925 100644
--- a/packaging/gstreamer.spec
+++ b/packaging/gstreamer.spec
@@ -62,7 +62,7 @@
 
 Name:           %{_name}
 Version:        1.20.0
-Release:        58
+Release:        59
 Summary:        Streaming-Media Framework Runtime
 License:        LGPL-2.0+
 Group:          Multimedia/Framework
diff --git a/subprojects/gst-plugins-base/gst/subparse/samiparse.c b/subprojects/gst-plugins-base/gst/subparse/samiparse.c
index 5227247f13..15820a1dfd 100644
--- a/subprojects/gst-plugins-base/gst/subparse/samiparse.c
+++ b/subprojects/gst-plugins-base/gst/subparse/samiparse.c
@@ -626,7 +626,7 @@ html_context_handle_element (HtmlContext * ctxt,
     }
     g_strfreev (attrs);
 
-    attrs = g_new0 (gchar *, count * 2);
+    attrs = g_new0 (gchar *, count * 2 + 1);
 
     for (i = 0; i < count; i++) {
       gchar *attr_name = NULL, *attr_value = NULL;