[gstreamer-bugs] [Bug 638107] [mp4mux] cover art lost when transcoding from flac to mp4

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=638107
  GStreamer | gst-plugins-bad | git

--- Comment #3 from Antonio Frediani <antonio.frediani at inwind.it> 2010-12-28 08:29:55 UTC ---
Hi Thiago,
thanks for the excellent information.
You're right about the type field of GST_TAG_IMAGE. However, I didn't find a
way to specify that information within the mp4 file itself, as it seems to me
that the metadata blocks only carry image type (jpg, png...) and size. 
Do you know of any more detailed information about the mp4 metadata format, or
have any proposal for the implementation of this part?

I have tried the tests you suggested but the results were quite different from
yours.
Flacdec correctly manages to print the metadata:

FOUND TAG      : found by element "flacdec0".
           image: buffer of 27732 bytes, type: image/jpeg, width=(int)305,
height=(int)350, image-type=(GstTagImageType)GST_TAG_IMAGE_TYPE_FRONT_COVER
FOUND TAG      : found by element "flacdec0".
     audio codec: FLAC
           album: Test Album
           title: Test Title
    track number: 1
extended comment: COMMENTS=Test Comment
          artist: Test Artist
            date: 2010-01-01
           genre: Classical
         comment: Test Comment

On the other hand, flacparse crashed on decoding the stream.

I tried several files with the same result. From the logs it seems a problem on
the cover image reading . I see the error:
flacparse gstflacparse.c:1077:gst_flac_parse_handle_picture:<flacparse0> Error
reading data
and when I tried to run it again on the same flac but having removed the cover
art it completed without error.

My flacdec is 0.10.25, while flacparse is 0.10.20. My system is a Ubuntu
Maverick. What vesion di you use? Do you want me to try another version?

I'm attaching an auto-generated flac file I used for test. I also have the
debug log of the flacparse but I don't think I should attach it to this bug.
Rather, if you think it necessary I will open another bug on flacparse.

Please let me know if you need any further information or tests

Thanks in advance,
Antonio

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.