commit 56b87f3a0fa1fe591a4f2ec4519ff08c765cd490
author Michael Stefaniuc <mstefani@redhat.de> Sun Oct 15 01:03:58 2006 +0200
committer Alexandre Julliard <julliard@winehq.org> Mon Oct 16 10:52:48 2006 +0200
tree 199c237278e4f667279fe7636569ed8bda16c3f6
parent 76b162067f9e5f2a373ddfab4151c2d8cb337fe5

janitorial: Remove redundant NULL checks before calling HeapFree wrappers.

dlls/rpcrt4/rpc_message.c

diff --git a/dlls/rpcrt4/rpc_message.c b/dlls/rpcrt4/rpc_message.c
index fb6187f..0120bf6 100644
--- a/dlls/rpcrt4/rpc_message.c
+++ b/dlls/rpcrt4/rpc_message.c
@@ -595,7 +595,7 @@
   status = RPC_S_OK;
 
 fail:
-  if (status != RPC_S_OK && *Header) {
+  if (status != RPC_S_OK) {
     RPCRT4_FreeHeader(*Header);
     *Header = NULL;
   }
@@ -765,9 +765,7 @@
   status = RPC_S_OK;
 
 fail:
-  if (hdr) {
-    RPCRT4_FreeHeader(hdr);
-  }
+  RPCRT4_FreeHeader(hdr);
   RPCRT4_CloseBinding(bind, conn);
   return status;
 }

dlls/winedos/module.c

diff --git a/dlls/winedos/module.c b/dlls/winedos/module.c
index e00b77e..24d6a73 100644
--- a/dlls/winedos/module.c
+++ b/dlls/winedos/module.c
@@ -273,7 +273,7 @@
         env_seg = par_env_seg;
     else
         env_seg=MZ_InitEnvironment(oldenv, filename);
-    if( alloc && oldenv)
+    if (alloc)
         FreeEnvironmentStringsA( oldenv);
 
     /* allocate memory for the executable */