[FFmpeg-devel] [PATCH 3/3] avformat/mxfdec: Read Apple private Content Light Level from MXF

[Harry Mallon]

> On 7 Sep 2020, at 10:46, Tomas Härdin <tjoppen at 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