Signed-off-by: Ziqing Hui zhui@codeweavers.com --- dlls/windowscodecs/ddsformat.c | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-)
+ if (This->pixel_data) HeapFree(GetProcessHeap(), 0, This->pixel_data);
It's not necessary to check pointers before calling HeapFree.
We should probably also lock the object while allocating and populating pixel_data.