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

[Carl Eugen Hoyos]

2018-08-31 1:43 GMT+02:00, Michael Niedermayer <michael at niedermayer.cc>:
> 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

Maybe Baptiste finds testclip_wrong_version_1_audio_stsd.mp4
or has a comment.

Carl Eugen