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 704294 - qtdemux: videos in Windows WRLE format don't work
qtdemux: videos in Windows WRLE format don't work
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
unspecified
Other Linux
: Normal normal
: 1.1.3
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2013-07-15 21:00 UTC by nils+bugzilla.gnome.org
Modified: 2013-07-19 19:22 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
partial patch (1.95 KB, patch)
2013-07-15 22:00 UTC, Olivier Crête 		committed Details | Review

Description nils+bugzilla.gnome.org 2013-07-15 21:00:38 UTC 
A file from PLOS does not play with GStreamer 0.10.36 on Debian: <http://www.ncbi.nlm.nih.gov/pmc/articles/PMC2991264/bin/pgen.1001219.s029.mov>

See also bug report for the Open Access Media Converter:
<https://github.com/erlehmann/open-access-media-importer/issues/69#issuecomment-19013499> – Daniel Mietchen asserts there that the bug affects 17 of 19 videos listed at <http://www.plosgenetics.org/article/info%3Adoi%2F10.1371%2Fjournal.pgen.1001219#section5>.
Comment 1 Olivier Crête 2013-07-15 22:00:23 UTC 
Created attachment 249250 [details] [review]
partial patch

Even with the attached patched, the palette for this sample is still wrong, on QuickTime and mplayer, it's grayscale with a black background... GStreamer shows strange colors, I assume the palette is incorrect.
Comment 2 Sebastian Dröge (slomo) 2013-07-16 08:41:09 UTC 
Maybe it is supposed to be grayscale? avplay shows it grayscale too.

There's some interesting grayscale logic in libavformat/mov.c around the palette code, maybe we should replicate that
Comment 3 Tim-Philipp Müller 2013-07-19 18:51:13 UTC 
Olivier: I think it looks right now, after:

 commit 4c97701650696352ce9b0db3e90f34c90ff9baef
 Author: Wim Taymans <wim.taymans@collabora.co.uk>
 Date:   Wed Jul 17 15:10:00 2013 +0200

    qtdemux: extract the palette from stsd
    
    Sometimes a palette is inside the stsd, extract it instead of always using
    the default one
Comment 4 Olivier Crête 2013-07-19 19:01:25 UTC 
Thanks Wim!

Author: Olivier Crête <olivier.crete@collabora.com>
Date:   Mon Jul 15 17:55:19 2013 -0400

    qtdemux: Add WRLE support