> 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".

Reply via email to