After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 690358 - matroskademux: empty string on taglist field warning
matroskademux: empty string on taglist field warning
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
1.x
Other Linux
: Normal normal
: 1.1.1
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-12-17 15:50 UTC by Jens Georg
Modified: 2012-12-17 23:15 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Jens Georg 2012-12-17 15:50:00 UTC 
Well, here you go:

File is http://blender-mirror.kino3d.org/mango/download.blender.org/demo/movies/ToS/tears_of_steel_1080p.mkv

gst-discoverer-1.0 file:///home/jens/Videos/tears_of_steel_1080p.mkv
Analyzing file:///home/jens/Videos/tears_of_steel_1080p.mkv

(gst-discoverer-1.0:14749): GStreamer-WARNING **: Trying to set empty string on taglist field 'title'. Please file a bug.

(gst-discoverer-1.0:14749): GStreamer-WARNING **: Trying to set empty string on taglist field 'artist'. Please file a bug.

(gst-discoverer-1.0:14749): GStreamer-WARNING **: Trying to set empty string on taglist field 'composer'. Please file a bug.

(gst-discoverer-1.0:14749): GStreamer-WARNING **: Trying to set empty string on taglist field 'genre'. Please file a bug.
Done discovering file:///home/jens/Videos/tears_of_steel_1080p.mkv

Topology:
  container: Matroska
    audio: MPEG-4 AAC
    video: Theora

Properties:
  Duration: 0:12:14.166687500
  Seekable: yes
  Tags: 
      extended comment: SYNOPSIS=
      date: 1901-01-01
      container format: Matroska
      audio codec: MPEG-4 AAC audio
      language code: en
      encoder: Xiph.Org libtheora 1.1 20090822 (Thusnelda)
      encoder version: 3
      video codec: Theora
      bitrate: 6100000
      nominal bitrate: 6100000


Backtrace for first warning:

+ Trace 231300

  • #0 g_logv
    at gmessages.c line 853
  • #1 g_log
    at gmessages.c line 1003
  • #2 gst_structure_set_field
    at gststructure.c line 784
  • #3 gst_structure_id_set_value_internal
    at gststructure.c line 470
  • #4 gst_structure_id_set_value
    at gststructure.c line 492
  • #5 gst_tag_list_add_value_internal
    at gsttaglist.c line 1054
  • #6 gst_tag_list_add_valist_values
    at gsttaglist.c line 1306
  • #7 gst_tag_list_add_values
    at gsttaglist.c line 1218
  • #8 gst_matroska_read_common_parse_metadata_id_simple_tag
    at matroska-read-common.c line 2052
  • #9 gst_matroska_read_common_parse_metadata_id_tag
    at matroska-read-common.c line 2326
  • #10 gst_matroska_read_common_parse_metadata
    at matroska-read-common.c line 2445
  • #11 gst_matroska_demux_parse_id
    at matroska-demux.c line 4314
  • #12 gst_matroska_demux_parse_contents_seekentry
    at matroska-demux.c line 3853
  • #13 gst_matroska_demux_parse_contents
    at matroska-demux.c line 3902
  • #14 gst_matroska_demux_parse_id
    at matroska-demux.c line 4332
  • #15 gst_matroska_demux_loop
    at matroska-demux.c line 4458
  • #16 gst_task_func
    at gsttask.c line 316
  • #17 g_thread_pool_thread_proxy
    at gthreadpool.c line 309
  • #18 g_thread_proxy
    at gthread.c line 798
  • #19 start_thread
    at pthread_create.c line 308
  • #20 clone
    at ../sysdeps/unix/sysv/linux/x86_64/clone.S line 112
  • #21 ?? 






Comment 1


Tim-Philipp Müller





          2012-12-17 23:04:31 UTC
        



Thanks for the bug report, should be fixed now:

 commit f4cb0c43150879b11e61027f57f337b4f57d21e5
 Author: Tim-Philipp Müller <tim@centricular.net>
 Date:   Mon Dec 17 22:55:12 2012 +0000

    matroskademux: skip empty tags
    
    instead of trying to add tags with empty strings, which
    causes criticals at runtime.
    
    https://bugzilla.gnome.org/show_bug.cgi?id=690358






Comment 2


Tim-Philipp Müller





          2012-12-17 23:15:33 UTC
        



Not needed on 1.0 branch, only in master.