From e3143703e9782b3622ed747a9c552a41b1fcaede Mon Sep 17 00:00:00 2001 From: Niklas Haas Date: Sat, 22 Apr 2023 21:36:18 +0200 Subject: [PATCH] avfilter/vf_libplacebo: strip ICC profiles on CSP change Not doing so is an obvious oversight - the ICC profile is tied to the original colorspace, so if we change it, we should definitely strip this information. We should probably also have an extra option to control whether the ICC profile should be stripped, ignored, or applied, but for now this fixes an existing bug. --- libavfilter/vf_libplacebo.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavfilter/vf_libplacebo.c b/libavfilter/vf_libplacebo.c index ba852de08d..66929223dd 100644 --- a/libavfilter/vf_libplacebo.c +++ b/libavfilter/vf_libplacebo.c @@ -535,6 +535,7 @@ static int filter_frame(AVFilterLink *link, AVFrame *in) if (changed_csp) { av_frame_remove_side_data(out, AV_FRAME_DATA_MASTERING_DISPLAY_METADATA); av_frame_remove_side_data(out, AV_FRAME_DATA_CONTENT_LIGHT_LEVEL); + av_frame_remove_side_data(out, AV_FRAME_DATA_ICC_PROFILE); } if (s->apply_dovi || changed_csp) { av_frame_remove_side_data(out, AV_FRAME_DATA_DOVI_RPU_BUFFER);