Michael Stefaniuc : windowscodecs: Remove a redundant NULL check before HeapFree().

Alexandre Julliard julliard at winehq.org
Sun Jul 10 08:57:09 CDT 2016


Module: wine
Branch: master
Commit: da5dfe7e80989c4dbdeb2fc3a933b8cc8fe66609
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=da5dfe7e80989c4dbdeb2fc3a933b8cc8fe66609

Author: Michael Stefaniuc <mstefani at redhat.de>
Date:   Fri Jul  8 10:06:02 2016 +0200

windowscodecs: Remove a redundant NULL check before HeapFree().

Signed-off-by: Michael Stefaniuc <mstefani at redhat.de>
Signed-off-by: Vincent Povirk <vincent at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/windowscodecs/icnsformat.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/dlls/windowscodecs/icnsformat.c b/dlls/windowscodecs/icnsformat.c
index d4c5004..552fa39 100644
--- a/dlls/windowscodecs/icnsformat.c
+++ b/dlls/windowscodecs/icnsformat.c
@@ -175,8 +175,7 @@ static ULONG WINAPI IcnsFrameEncode_Release(IWICBitmapFrameEncode *iface)
             This->encoder->outstanding_commits--;
             LeaveCriticalSection(&This->encoder->lock);
         }
-        if (This->icns_image != NULL)
-            HeapFree(GetProcessHeap(), 0, This->icns_image);
+        HeapFree(GetProcessHeap(), 0, This->icns_image);
 
         IWICBitmapEncoder_Release(&This->encoder->IWICBitmapEncoder_iface);
         HeapFree(GetProcessHeap(), 0, This);




More information about the wine-cvs mailing list