Michael Stefaniuc : mscoree: Remove superfluous NULL pointer checks before free (Smatch).

Alexandre Julliard julliard at winehq.org
Mon Jan 16 13:01:29 CST 2012


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

Author: Michael Stefaniuc <mstefani at redhat.de>
Date:   Mon Jan 16 00:17:31 2012 +0100

mscoree: Remove superfluous NULL pointer checks before free (Smatch).

---

 dlls/mscoree/corruntimehost.c |   12 ++++--------
 1 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/dlls/mscoree/corruntimehost.c b/dlls/mscoree/corruntimehost.c
index 5042f20..00966ee 100644
--- a/dlls/mscoree/corruntimehost.c
+++ b/dlls/mscoree/corruntimehost.c
@@ -609,14 +609,10 @@ static HRESULT WINAPI CLRRuntimeHost_ExecuteInDefaultAppDomain(ICLRRuntimeHost*
     }
 
 cleanup:
-    if(filenameA)
-        HeapFree(GetProcessHeap(), 0, filenameA);
-    if(classA)
-        HeapFree(GetProcessHeap(), 0, classA);
-    if(argsA)
-        HeapFree(GetProcessHeap(), 0, argsA);
-    if(methodA)
-        HeapFree(GetProcessHeap(), 0, methodA);
+    HeapFree(GetProcessHeap(), 0, filenameA);
+    HeapFree(GetProcessHeap(), 0, classA);
+    HeapFree(GetProcessHeap(), 0, argsA);
+    HeapFree(GetProcessHeap(), 0, methodA);
 
     return hr;
 }




More information about the wine-cvs mailing list