commit a87520036f7b9ca8a7292fc8fc53df1fd4ea6ecb
author Michael Stefaniuc <mstefani@redhat.de> Wed May 10 00:33:53 2006 +0200
committer Alexandre Julliard <julliard@winehq.org> Wed May 10 11:31:41 2006 +0200
tree 76933dad43bc722bda2903d03e0f1197a3caa84f
parent 9d4960bfcb927644503861ae627cb79b9aceb81c

janitorial: Remove redundant NULL pointer checks before HeapFree'ing them.

Some HeapFree's are hidden behind macros. Found by smatch.

dlls/secur32/secur32.c

diff --git a/dlls/secur32/secur32.c b/dlls/secur32/secur32.c
index ebe7e2e..664f901 100644
--- a/dlls/secur32/secur32.c
+++ b/dlls/secur32/secur32.c
@@ -666,8 +666,7 @@
     {
         LIST_FOR_EACH_ENTRY(provider, &providerTable->table, SecureProvider, entry)
         {
-            if (provider->moduleName)
-                SECUR32_FREE(provider->moduleName);
+            SECUR32_FREE(provider->moduleName);
             if (provider->lib)
                 FreeLibrary(provider->lib);
         }
@@ -689,7 +688,7 @@
  */
 SECURITY_STATUS WINAPI FreeContextBuffer(PVOID pv)
 {
-    if (pv) SECUR32_FREE(pv);
+    SECUR32_FREE(pv);
 
     return SEC_E_OK;
 }