[FFmpeg-devel] [PATCH] avformat: Add simple ACLR atom reading to set the color range

Kevin Wheatley kevin.j.wheatley at gmail.com
Tue Feb 17 12:58:06 CET 2015


Oops, I missed that when I allowed the atom to be added to the end of
the extradata, updated.

I'm not sure if logging as an error is appropriate in the case when
the extradata is not large enough, for this to occur the file will
have been truncated somehow and the mov_read_extradata function will
have logged a warning and returned 0. Which at first glance seams
wrong - should mov_read_extradata return something other than 0 in
this case?

Either way it does mean the code could read the wrong value, but
should not crash with the guard in place.

Thanks

Kevin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Add-simple-ACLR-atom-reading-to-set-the-color-range-.patch
Type: text/x-patch
Size: 2800 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150217/0ae6bfad/attachment.bin>


More information about the ffmpeg-devel mailing list