[gstreamer-bugs] [Bug 407349] New: id3demux reports incorrect year when TDAT is present

[GStreamer (bugzilla.gnome.org)]

Do not reply to this via email (we are currently unable to handle email
responses and they get discarded).  You can add comments to this bug at
http://bugzilla.gnome.org/show_bug.cgi?id=407349

  GStreamer | gst-plugins-good | Ver: 0.10.3

           Summary: id3demux reports incorrect year when TDAT is present
           Product: GStreamer
           Version: 0.10.3
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: sdc at sadach.org
         QAContact: gstreamer-bugs at lists.sourceforge.net
     GNOME version: Unspecified
   GNOME milestone: Unspecified


Seems id3demux naively converts TDAT fields to TDRC, with the effect
that the value of TDAT is interpreted as a year. This is only
noticeable when the TDAT tag precedes the TYER field in the source
file. I'll attach a couple of files to demonstrate this.


This case is fine.

$ id3info tyer-before-tdat.mp3

*** Tag information for tyer-before-tdat.mp3
=== TYER (Year): 1977
=== TDAT (Date): 2306
*** mp3 info
MPEG1/layer III
Bitrate: 64KBps
Frequency: 44KHz

$ gst-launch-0.10 filesrc location=tyer-before-tdat.mp3 ! id3demux ! \
  fakesink -t

Setting pipeline to PAUSED ...
Pipeline is PREROLLING ...
FOUND TAG      : found by element "id3demux0".
            date: 1977-01-01
Pipeline is PREROLLED ...
Setting pipeline to PLAYING ...
New clock: GstSystemClock
Got EOS from element "pipeline0".
Execution ended after 950000 ns.
Setting pipeline to PAUSED ...
Setting pipeline to READY ...
Setting pipeline to NULL ...
FREEING pipeline ...


I'd expect to get the same answer in this case, but I don't.

$ id3info tdat-before-tyer.mp3

*** Tag information for tdat-before-tyer.mp3
=== TDAT (Date): 2306
=== TYER (Year): 1977
*** mp3 info
MPEG1/layer III
Bitrate: 64KBps
Frequency: 44KHz

$ gst-launch-0.10 filesrc location=tdat-before-tyer.mp3 ! id3demux ! \
  fakesink -t

Setting pipeline to PAUSED ...
Pipeline is PREROLLING ...
FOUND TAG      : found by element "id3demux0".
            date: 2306-01-01
Pipeline is PREROLLED ...
Setting pipeline to PLAYING ...
New clock: GstSystemClock
Got EOS from element "pipeline0".
Execution ended after 880000 ns.
Setting pipeline to PAUSED ...
Setting pipeline to READY ...
Setting pipeline to NULL ...
FREEING pipeline ...


-- 
Configure bugmail: http://bugzilla.gnome.org/userprefs.cgi?tab=email