commit 5ad7d858e0d6168429fca902500ac5b50f14dc30
author Michael Stefaniuc <mstefani@redhat.de> Thu Dec 23 17:06:43 2004 +0000
committer Alexandre Julliard <julliard@winehq.org> Thu Dec 23 17:06:43 2004 +0000
tree 5801e7f60e1982dfd7a54c450c5ab3c015cb8e6a
parent bf42021e06b6757db5b014a8cde443aada22a0a1

Do not check for non NULL pointer before HeapFree'ing it. It's
redundant.

dlls/wineps/init.c

diff --git a/dlls/wineps/init.c b/dlls/wineps/init.c
index a3ecdce..200d7cc 100644
--- a/dlls/wineps/init.c
+++ b/dlls/wineps/init.c
@@ -369,8 +369,7 @@
     TRACE("\n");
 
     HeapFree( PSDRV_Heap, 0, physDev->Devmode );
-    if(physDev->job.output)
-        HeapFree( PSDRV_Heap, 0, physDev->job.output );
+    HeapFree( PSDRV_Heap, 0, physDev->job.output );
     HeapFree( PSDRV_Heap, 0, physDev );
 
     return TRUE;
@@ -753,14 +752,10 @@
 closeprinter:
     ClosePrinter(hPrinter);
 cleanup:
-    if (ppdFileName)
-        HeapFree(PSDRV_Heap, 0, ppdFileName);
-    if (pi->FontSubTable)
-    	HeapFree(PSDRV_Heap, 0, pi->FontSubTable);
-    if (pi->FriendlyName)
-    	HeapFree(PSDRV_Heap, 0, pi->FriendlyName);
-    if (pi->Devmode)
-    	HeapFree(PSDRV_Heap, 0, pi->Devmode);
+    HeapFree(PSDRV_Heap, 0, ppdFileName);
+    HeapFree(PSDRV_Heap, 0, pi->FontSubTable);
+    HeapFree(PSDRV_Heap, 0, pi->FriendlyName);
+    HeapFree(PSDRV_Heap, 0, pi->Devmode);
 fail:
     HeapFree(PSDRV_Heap, 0, pi);
     if (ppd) unlink(ppd);