On Fri, Jun 10, 2016 at 4:56 AM, Anton Khirnov <[email protected]> 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 [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
