Michael Stefaniuc : mshtml: Remove redundant NULL check before HeapFree ( Smatch).

Alexandre Julliard julliard at winehq.org
Tue Oct 20 10:33:37 CDT 2009


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

Author: Michael Stefaniuc <mstefani at redhat.de>
Date:   Mon Oct 19 22:55:03 2009 +0200

mshtml: Remove redundant NULL check before HeapFree (Smatch).

---

 dlls/mshtml/htmllocation.c |    6 ++----
 1 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/dlls/mshtml/htmllocation.c b/dlls/mshtml/htmllocation.c
index 5db5267..e7b6649 100644
--- a/dlls/mshtml/htmllocation.c
+++ b/dlls/mshtml/htmllocation.c
@@ -256,10 +256,8 @@ static HRESULT WINAPI HTMLLocation_get_href(IHTMLLocation *iface, BSTR *p)
     ret = S_OK;
 
 cleanup:
-    if(buf)
-        HeapFree(GetProcessHeap(), 0, buf);
-    if(url_path)
-        HeapFree(GetProcessHeap(), 0, url_path);
+    HeapFree(GetProcessHeap(), 0, buf);
+    HeapFree(GetProcessHeap(), 0, url_path);
 
     return ret;
 }




More information about the wine-cvs mailing list