flvdemux: Don't write an empty string as a tag
authorVivia Nikolaidou <vivia@ahiru.eu>
Wed, 8 Apr 2020 16:49:00 +0000 (19:49 +0300)
committerVivia Nikolaidou <vivia@ahiru.eu>
Wed, 8 Apr 2020 17:22:51 +0000 (20:22 +0300)
To stop warnings like:

GStreamer-WARNING **: 19:47:48.186: Trying to set empty string on
taglist field 'encoder'. Please file a bug.

gst/flv/gstflvdemux.c

index 9342ca1..5ae117f 100644 (file)
@@ -402,6 +402,11 @@ gst_flv_demux_parse_metadata_item (GstFlvDemux * demux, GstByteReader * reader,
       s = FLV_GET_STRING (reader);
       if (s == NULL)
         goto error;
+      if (!strcmp (s, "")) {
+        /* Not strictly an error, just an empty string */
+        g_free (s);
+        break;
+      }
 
       GST_DEBUG_OBJECT (demux, "%s => (string) %s", tag_name, s);