winealsa.drv: Remove redundant NULL checks before free (Smatch).

d8ca6972 · Michael Stefaniuc · Alexandre Julliard · 036db280 · d8ca6972
Commit d8ca6972 authored Oct 11, 2011 by Michael Stefaniuc Committed by Alexandre Julliard Oct 11, 2011
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 12 deletions

mmdevdrv.c dlls/winealsa.drv/mmdevdrv.c +6 -12

No files found.
--- a/dlls/winealsa.drv/mmdevdrv.c
+++ b/dlls/winealsa.drv/mmdevdrv.c
@@ -1043,18 +1043,12 @@ static HRESULT WINAPI AudioClient_Initialize(IAudioClient *iface,
 exit:
    HeapFree(GetProcessHeap(), 0, sw_params);
    if(FAILED(hr)){
-        if(This->local_buffer){
-            HeapFree(GetProcessHeap(), 0, This->local_buffer);
-            This->local_buffer = NULL;
-        }
-        if(This->fmt){
-            CoTaskMemFree(This->fmt);
-            This->fmt = NULL;
-        }
-        if(This->vols){
-            HeapFree(GetProcessHeap(), 0, This->vols);
-            This->vols = NULL;
-        }
+        HeapFree(GetProcessHeap(), 0, This->local_buffer);
+        This->local_buffer = NULL;
+        CoTaskMemFree(This->fmt);
+        This->fmt = NULL;
+        HeapFree(GetProcessHeap(), 0, This->vols);
+        This->vols = NULL;
    }

    LeaveCriticalSection(&This->lock);