devenum: Check for null pointer in IMoniker_BindToObject().

Signed-off-by: Zebediah Figura <z.figura12@gmail.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

devenum: Check for null pointer in IMoniker_BindToObject().
8cd4d9fb · Zebediah Figura · Alexandre Julliard · a46e47f8 · 8cd4d9fb · 8cd4d9fb
Commit 8cd4d9fb authored Apr 28, 2018 by Zebediah Figura Committed by Alexandre Julliard Apr 30, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 0 deletions

mediacatenum.c dlls/devenum/mediacatenum.c +3 -0

devenum.c dlls/devenum/tests/devenum.c +3 -0

No files found.
--- a/dlls/devenum/mediacatenum.c
+++ b/dlls/devenum/mediacatenum.c
@@ -444,6 +444,9 @@ static HRESULT WINAPI DEVENUM_IMediaCatMoniker_BindToObject(IMoniker *iface, IBi

    TRACE("(%p)->(%p, %p, %s, %p)\n", This, pbc, pmkToLeft, debugstr_guid(riidResult), ppvResult);

+    if (!ppvResult)
+        return E_POINTER;
+
    VariantInit(&var);
    *ppvResult = NULL;


--- a/dlls/devenum/tests/devenum.c
+++ b/dlls/devenum/tests/devenum.c
@@ -134,6 +134,9 @@ static void test_devenum(IBindCtx *bind_ctx)
                    }
                }

+                hr = IMoniker_BindToObject(moniker, bind_ctx, NULL, &IID_IUnknown, NULL);
+                ok(hr == E_POINTER, "got %#x\n", hr);
+
                IPropertyBag_Release(prop_bag);
                IMoniker_Release(moniker);
            }