[gstreamer-bugs] [Bug 508291] New: id3demux should check language code in id3v2 comment tag for non-utf8 strings

[GStreamer (bugzilla.gnome.org)]

If you have any questions why you received this email, please see the text at
the end of this email. Replies to this email are NOT read, please see the text
at the end of this email. You can add comments to this bug at:
  http://bugzilla.gnome.org/show_bug.cgi?id=508291

  GStreamer | gst-plugins-good | Ver: HEAD CVS
           Summary: id3demux should check language code in id3v2 comment tag
                    for non-utf8 strings
           Product: GStreamer
           Version: HEAD CVS
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: ext-tommi.myohanen at nokia.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


The following warning is displayed with attached clip:
$ gst-launch playbin uri=file:///tmp/kpa.mp3 
Setting pipeline to PAUSED ...

(gst-launch-0.10:1341): GStreamer-WARNING **: Trying to set string field
'extended-comment' on structure, but string is not valid UTF-8. Please file a
bug.
Pipeline is PREROLLING ...
Pipeline is PREROLLED ...
Setting pipeline to PLAYING ...
New clock: GstAudioSinkClock

The warning is caused by strange language code field in id3v2 comment tag of
the file, so maybe id3demux should check this field too for non-utf8 strings.

Also, the warning comes from gst_structure_id_set_value() where non-utf8
strings are skipped, but only if gst-debug is enabled. To avoid confusion I
think enabling gst-debug should only add the debug prints and not change the
behaviour of the code.


-- 
See http://bugzilla.gnome.org/page.cgi?id=email.html for more info about why you received
this email, why you can't respond via email, how to stop receiving
emails (or reduce the number you receive), and how to contact someone
if you are having problems with the system.

You can add comments to this bug at http://bugzilla.gnome.org/show_bug.cgi?id=508291.