[Bug 731029] Incorrect reading of the GST_TAG_DATE_TIME field of a MPEG-4 AAC encoded file

[GStreamer (GNOME Bugzilla)]

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

Marinus Schraal <mschraal at gnome.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |mschraal at gnome.org

--- Comment #14 from Marinus Schraal <mschraal at gnome.org> ---
I was just looking into this. The problem here is that the tags returned are
different but imply to be the same:

https://cgit.freedesktop.org/gstreamer/gstreamer/tree/gst/gsttaglist.c#n137

lists 2 content created related tags (DATE & DATE_TIME), their descriptions are
the same and only say they return different time objects.

qtdemux fills these 2 tags with different values depending on what it
encounters. However it specifically states in
https://cgit.freedesktop.org/gstreamer/gst-plugins-good/tree/gst/isomp4/qtdemux.c#n12596
that we should take the mhvd field value as less important than any specific
tags.

Now the mhvd field gets tagged in gstreamer as DATE_TIME and the real tags get
tagged as DATE only. 

So as far as I can see the commenter in the qtdemux code above meant that tag
to be bottom of the pack, it just doesn't get handled that way.

As a solution I think the other fields should be converted to a full datetime
to get the desired (and expected) behaviour. If there are no other tags
present, only then the mvhd tag gets returned.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.