samiparse: Check that the string has a non-zero length before overwriting the last...

https://bugzilla.gnome.org/show_bug.cgi?id=777502

diff --git a/gst/subparse/samiparse.c b/gst/subparse/samiparse.c
index 517e959a91ed31ad72f20380639f63440a07f0f5..377c6d730c6aab3adb9e992ca91cfdd915ad2371 100644 (file)
--- a/gst/subparse/samiparse.c
+++ b/gst/subparse/samiparse.c
@@ -504,7 +504,8 @@ html_context_handle_element (HtmlContext * ctxt,
     }
 
     length = strlen (attr_value);
-    if (attr_value[length - 1] == '"' || attr_value[length - 1] == '\'') {
+    if (length > 0 && (attr_value[length - 1] == '"'
+            || attr_value[length - 1] == '\'')) {
       attr_value[length - 1] = '\0';
     }