[FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized QuickTime video properly

Mats Peterson matsp888 at yahoo.com
Sat Dec 12 12:17:00 CET 2015

Obviously that private data is cropped in some way then, since the minimum size of a video sample description in QuickTime video is 86 bytes.


Mats Peterson

----- Original Message -----
From: Michael Niedermayer <michaelni at gmx.at>
To: FFmpeg development discussions and patches <ffmpeg-devel at ffmpeg.org>
Sent: Saturday, December 12, 2015 11:58 AM
Subject: Re: [FFmpeg-devel] [PATCH] matroskadec.c: Handle palettized	QuickTime video properly

On Fri, Dec 11, 2015 at 11:32:13AM +0000, Mats Peterson wrote:

> OK, here is an improved version of my previous patch of matroskadec.c that manages the palette itself without calling code in mov.c. I have borrowed some code from it, though, and I'm including qtpalette.h. One thing I don't understand is why the minimum private data size for V_QUICKTIME has been the mysterioius value 21 so far, since 86 is the minimum size for a video sample description without a palette. So I changed that. As I said before, I have also changed the minimum private data size for A_QUICKTIME from 86 to 36, since 36 is the size of a version 0 sound sample description.
> Mats -- 
> Mats Peterson
> http://matsp888.no-ip.org/~mats/

>  matroskadec.c |   94 ++++++++++++++++++++++++++++++++++++++++++++++++++++++----
>  1 file changed, 89 insertions(+), 5 deletions(-)
> 6d1e32a75c7d8de7dfb02756a135ad7855c49b90  matroskadec.c.diff
> --- matroskadec.c.orig    2015-12-08 11:01:40.640478749 +0100
> +++ matroskadec.c    2015-12-11 12:00:13.000000000 +0100

this breaks demuxing of the video stream from
(google should find a url for this)


Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The worst form of inequality is to try to make unequal things equal.
-- Aristotle

ffmpeg-devel mailing list
ffmpeg-devel at ffmpeg.org

More information about the ffmpeg-devel mailing list