> On 7 Sep 2020, at 10:46, Tomas Härdin <tjop...@acc.umu.se> wrote: > > mån 2020-08-31 klockan 20:07 +0100 skrev Harry Mallon: >> * As embedded by Apple Compressor > > This needs a sample since it isn't part of any official spec, so that > we can have a test for this. > >> + if (IS_KLV_KEY(uid, mxf_coll_apple_max_cll)) { >> + if (!descriptor->coll) { >> + descriptor->coll = >> av_content_light_metadata_alloc(&descriptor->coll_size); >> + if (!descriptor->coll) >> + return AVERROR(ENOMEM); >> + } > > Duplicated allocation here as well. See my comment on PATCH 1/3. >
Sample and FATE test in v2. Best, Harry _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".