[FFmpeg-cvslog] avfilter/vf_libplacebo: strip ICC profiles on CSP change

[Niklas Haas]

ffmpeg | branch: master | Niklas Haas <git at haasn.dev> | Sat Apr 22 21:36:18 2023 +0200| [e3143703e9782b3622ed747a9c552a41b1fcaede] | committer: Niklas Haas

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.

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=e3143703e9782b3622ed747a9c552a41b1fcaede
---

 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);