mirrors/gstreamer
mirrors/gstreamer
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gstreamer.git synced 2025-01-08 00:16:13 +00:00
Code Issues 1 Projects Releases Wiki Activity

gstvalue: don't write to const char *

Browse source 
Our various deserializing functions require NULL terminators
to not over consume substrings (eg fields of an array). Instead
of writing a NULL terminator to the passed-in string, which may
result in segfaults, make a copy of the substring we're interested
in.

Fixes https://gitlab.freedesktop.org/gstreamer/gstreamer/-/issues/446

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/674>
This commit is contained in:
Mathieu Duponchelle 2020-10-13 01:19:47 +02:00 committed by Tim-Philipp Müller
parent fa2f853013
commit e846ef2221
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
gst/gstvalue.c
View file

@ -2577,8 +2577,7 @@ _priv_gst_value_parse_value (gchar * str,
if (G_UNLIKELY (!_priv_gst_value_parse_string (s, &value_end, &s, TRUE)))
return FALSE;
/* Set NULL terminator for deserialization */
c = *value_end;
*value_end = '\0';
value_s = g_strndup (value_s, value_end - value_s);
for (i = 0; i < G_N_ELEMENTS (try_types); i++) {
g_value_init (value, try_types[i]);
@ -2594,14 +2593,13 @@ _priv_gst_value_parse_value (gchar * str,
(type != G_TYPE_STRING))))
return FALSE;
/* Set NULL terminator for deserialization */
c = *value_end;
*value_end = '\0';
value_s = g_strndup (value_s, value_end - value_s);
ret = gst_value_deserialize (value, value_s);
if (G_UNLIKELY (!ret))
g_value_unset (value);
}
*value_end = c;
g_free (value_s);
}
*after = s;