From 2a9a42fd23bfdb13512e58f8854fcbb1cdb5aaf2 Mon Sep 17 00:00:00 2001 From: Vadim Zeitlin Date: Tue, 29 Sep 2015 00:44:08 +0200 Subject: [PATCH] Don't pretend that we implement IWICBitmapSource::CopyPalette() We don't and it's unclear how to do it, so just return WINCODEC_ERR_PALETTEUNAVAILABLE instead of pretending that we copied the palette when we didn't. See #16625. --- src/msw/graphicsd2d.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/src/msw/graphicsd2d.cpp b/src/msw/graphicsd2d.cpp index f0b3e9debe..061582fe93 100644 --- a/src/msw/graphicsd2d.cpp +++ b/src/msw/graphicsd2d.cpp @@ -1375,10 +1375,9 @@ public: return S_OK; } - HRESULT STDMETHODCALLTYPE CopyPalette(__RPC__in_opt IWICPalette *palette) wxOVERRIDE + HRESULT STDMETHODCALLTYPE CopyPalette(__RPC__in_opt IWICPalette* WXUNUSED(palette)) wxOVERRIDE { - palette = NULL; - return S_OK; + return WINCODEC_ERR_PALETTEUNAVAILABLE; } HRESULT STDMETHODCALLTYPE CopyPixels(