Michael Stefaniuc : mmdevapi: Remove superfluous NULL pointer check before HeapFree (Smatch).

Alexandre Julliard julliard at winehq.org
Mon Apr 16 13:35:44 CDT 2012


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

Author: Michael Stefaniuc <mstefani at redhat.de>
Date:   Sun Apr 15 16:31:57 2012 +0200

mmdevapi: Remove superfluous NULL pointer check before HeapFree (Smatch).

---

 dlls/mmdevapi/devenum.c |    3 +--
 1 files changed, 1 insertions(+), 2 deletions(-)

diff --git a/dlls/mmdevapi/devenum.c b/dlls/mmdevapi/devenum.c
index 866aa06..075313c 100644
--- a/dlls/mmdevapi/devenum.c
+++ b/dlls/mmdevapi/devenum.c
@@ -301,8 +301,7 @@ static MMDevice *MMDevice_Create(WCHAR *name, GUID *id, EDataFlow flow, DWORD st
     }else if(cur->ref > 0)
         WARN("Modifying an MMDevice with postitive reference count!\n");
 
-    if(cur->drv_id)
-        HeapFree(GetProcessHeap(), 0, cur->drv_id);
+    HeapFree(GetProcessHeap(), 0, cur->drv_id);
     cur->drv_id = name;
 
     cur->flow = flow;




More information about the wine-cvs mailing list