[FFmpeg-devel] [PATCH] lavf/mov: Fix PCM audio w/ bit depth > 16

Michael Niedermayer michael at niedermayer.cc
Fri Aug 31 02:43:47 EEST 2018


On Mon, Aug 27, 2018 at 12:59:28PM -0700, John Stebbins wrote:
> This type of audio is defined by the QT spec, but can be found in
> non-QT branded files in the wild.
> 
> Fixes ticket #7376
> ---
>  libavformat/mov.c | 14 ++++++++++++++
>  1 file changed, 14 insertions(+)

So IIUC this is supporting a case that is slightly violating the spec ?
Iam asking as i was wondering if it makes sense to check a few more
things. For example the size could be checked if it actually is
large enough to contain a version=1 header

either way i think this patch should be ok

[...]

thx
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The misfortune of the wise is better than the prosperity of the fool.
-- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180831/91c29936/attachment.sig>


More information about the ffmpeg-devel mailing list