diff --git a/gst/gststructure.c b/gst/gststructure.c index 3aac67654f..ad42d3aad3 100644 --- a/gst/gststructure.c +++ b/gst/gststructure.c @@ -1922,8 +1922,11 @@ gst_structure_parse_string (gchar * s, gchar ** end, gchar ** next, while (*s != '"') { if (G_UNLIKELY (*s == 0)) return FALSE; - if (G_UNLIKELY (*s == '\\')) + if (G_UNLIKELY (*s == '\\')) { s++; + if (G_UNLIKELY (*s == 0)) + return FALSE; + } *w = *s; w++; s++; @@ -1935,8 +1938,11 @@ gst_structure_parse_string (gchar * s, gchar ** end, gchar ** next, while (*s != '"') { if (G_UNLIKELY (*s == 0)) return FALSE; - if (G_UNLIKELY (*s == '\\')) + if (G_UNLIKELY (*s == '\\')) { s++; + if (G_UNLIKELY (*s == 0)) + return FALSE; + } s++; } s++; diff --git a/tests/check/gst/gststructure.c b/tests/check/gst/gststructure.c index 1dd7cd677c..60e532c565 100644 --- a/tests/check/gst/gststructure.c +++ b/tests/check/gst/gststructure.c @@ -174,6 +174,20 @@ GST_START_TEST (test_from_string) ASSERT_WARNING (structure = gst_structure_from_string (s, NULL)); fail_if (structure == NULL, "Could not get structure from string %s", s); gst_structure_free (structure); + + /* make sure we handle \ as last character in various things, run with valgrind */ + s = "foo,test=\"foobar\\"; + structure = gst_structure_from_string (s, NULL); + fail_unless (structure == NULL); + s = "\\"; + structure = gst_structure_from_string (s, NULL); + fail_unless (structure == NULL); + s = "foobar,test\\"; + structure = gst_structure_from_string (s, NULL); + fail_unless (structure == NULL); + s = "foobar,test=(string)foo\\"; + structure = gst_structure_from_string (s, NULL); + fail_unless (structure == NULL); } GST_END_TEST;