[FFmpeg-devel] [PATCH] tiffdec: support embedded ICC profiles

Derek Buitenhuis derek.buitenhuis at gmail.com
Mon Jan 13 18:06:39 EET 2020


On 10/01/2020 22:06, Lynne wrote:
> Patch attached.
> 
> Very widespread, every NASA TIFF image has an ICC profile embedded, and its almost never sRGB, so this is really needed for proper color display.

[...]

> ---
>  libavcodec/tiff.c | 13 +++++++++++++
>  libavcodec/tiff.h |  1 +
>  2 files changed, 14 insertions(+)

> +    case TIFF_ICC_PROFILE:
> +        if (count >= INT_MAX || count <= 0)
> +            return AVERROR_INVALIDDATA;

Should this be > instead of >=?

Otherwise, LGTM.

- Derek


More information about the ffmpeg-devel mailing list