On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov <an...@khirnov.net> wrote:
> Quoting Vittorio Giovara (2016-06-10 00:55:19)
>> This function needs to return false, or data in the additional tables
>> will be skipped, and the decoder will not be able to decode frames
>> associated with them.
>>
>> ---
>> libavformat/mov.c | 2 ++
>> 1 file changed, 2 insertions(+)
>>
>> diff --git a/libavformat/mov.c b/libavformat/mov.c
>> index 9e2d0e2..9a7a0bc 100644
>> --- a/libavformat/mov.c
>> +++ b/libavformat/mov.c
>> @@ -1774,6 +1774,8 @@ static int mov_skip_multiple_stsd(MOVContext *c,
>> AVIOContext *pb,
>> (codec_tag == AV_RL32("hvc1") ||
>> codec_tag == AV_RL32("hev1") ||
>> (codec_tag != format &&
>> + // prores is allowed to have differing data format and codec tag
>> + codec_tag != AV_RL32("apcn") && codec_tag != AV_RL32("apch") &&
>> (c->fc->video_codec_id ? video_codec_id != c->fc->video_codec_id
>> : codec_tag != MKTAG('j','p','e','g'))))) {
>> /* Multiple fourcc, we skip JPEG. This is not correct, we should
>> --
>> 2.8.3
>
> Sure would be nice to have a FATE test.
i can make one for the h264 one
--
Vittorio
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
