On 12/13/2015 11:06 AM, Paul B Mahol wrote:
On 12/13/15, Mats Peterson <matsp888-at-yahoo....@ffmpeg.org> wrote:
On Sat, 12 Dec 2015, Paul B Mahol wrote:
On 12/12/15, Mats Peterson <matsp888-at-yahoo....@ffmpeg.org> wrote:
On Sat, 12 Dec 2015, Michael Niedermayer wrote:
On Sat, Dec 12, 2015 at 11:17:00AM +0000, Mats Peterson wrote:
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.
FFmpeg tries to support all kind of odd and broken files, so even if
the file is invalid, continuing to support it would be better
Personally I don't see why it should support broken files, but your
mileage may vary, of course. So just keep the value 21 then.
Have you provided file that doesn't work?
I got a sample Matroska file from Michael Niedermayer with V_QUICKTIME
video and with a private data of only 21 bytes, when it should be at least
86 bytes, which is the minimum size for a video sample description in
QuickTime.
What about non-broken files?
Mats
--
Mats Peterson
http://matsp888.no-ip.org/~mats/
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
I'm sorry, I don't understand what you're after. Of course I have tried
this patch with several test files.
Mats
--
Mats Peterson
http://matsp888.no-ip.org/~mats/
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
