mmdevapi: Skip device if GetId fails (Coverity).

Signed-off-by: André Hentschel <nerv@dawncrow.de> Signed-off-by: Andrew Eikum <aeikum@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

mmdevapi: Skip device if GetId fails (Coverity).
bbc42a23 · André Hentschel · Alexandre Julliard · fafab917 · bbc42a23
Commit bbc42a23 authored Dec 22, 2015 by André Hentschel Committed by Alexandre Julliard Dec 23, 2015
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

devenum.c dlls/mmdevapi/devenum.c +7 -1

No files found.
--- a/dlls/mmdevapi/devenum.c
+++ b/dlls/mmdevapi/devenum.c
@@ -1065,9 +1065,15 @@ static HRESULT WINAPI MMDevEnum_GetDevice(IMMDeviceEnumerator *iface, const WCHA
    for (i = 0; i < MMDevice_count; ++i)
    {
+        HRESULT hr;
        WCHAR *str;
        dev = &MMDevice_head[i]->IMMDevice_iface;
-        IMMDevice_GetId(dev, &str);
+        hr = IMMDevice_GetId(dev, &str);
+        if (FAILED(hr))
+        {
+            WARN("GetId failed: %08x\n", hr);
+            continue;
+        }
        if (str && !lstrcmpW(str, name))
        {