Nikolay Sivov : ole32/antimoniker: Fix Enum() return error code.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: d4cb70107d56416f19e44a962f732cf6cd696f86
URL:    https://gitlab.winehq.org/wine/wine/-/commit/d4cb70107d56416f19e44a962f732cf...

Author: Nikolay Sivov 
Date:   Sat May 27 10:39:15 2023 +0200

ole32/antimoniker: Fix Enum() return error code.

---

dlls/ole32/antimoniker.c   | 10 +++-------
 dlls/ole32/tests/moniker.c |  4 +++-
 2 files changed, 6 insertions(+), 8 deletions(-)

diff --git a/dlls/ole32/antimoniker.c b/dlls/ole32/antimoniker.c
index e4ee4a729d4..855bd47fb27 100644
--- a/dlls/ole32/antimoniker.c
+++ b/dlls/ole32/antimoniker.c
@@ -294,16 +294,12 @@ AntiMonikerImpl_ComposeWith(IMoniker* iface, IMoniker* pmkRight,
         return CreateGenericComposite(iface,pmkRight,ppmkComposite);
 }

-/******************************************************************************
- *        AntiMoniker_Enum
- ******************************************************************************/
-static HRESULT WINAPI
-AntiMonikerImpl_Enum(IMoniker* iface,BOOL fForward, IEnumMoniker** ppenumMoniker)
+static HRESULT WINAPI AntiMonikerImpl_Enum(IMoniker *iface, BOOL forward, IEnumMoniker **ppenumMoniker)
 {
-    TRACE("(%p,%d,%p)\n",iface,fForward,ppenumMoniker);
+    TRACE("%p, %d, %p.\n", iface, forward, ppenumMoniker);

if (ppenumMoniker == NULL)
-        return E_POINTER;
+        return E_INVALIDARG;

*ppenumMoniker = NULL;

diff --git a/dlls/ole32/tests/moniker.c b/dlls/ole32/tests/moniker.c
index a507c63171c..9c583d27c47 100644
--- a/dlls/ole32/tests/moniker.c
+++ b/dlls/ole32/tests/moniker.c
@@ -3037,7 +3037,9 @@ static void test_anti_moniker(void)
     ok(!enummoniker, "Unexpected pointer.\n");

hr = IMoniker_Enum(moniker, FALSE, NULL);
-    todo_wine
+    ok(hr == E_INVALIDARG, "Unexpected hr %#lx.\n", hr);
+
+    hr = IMoniker_Enum(moniker, TRUE, NULL);
     ok(hr == E_INVALIDARG, "Unexpected hr %#lx.\n", hr);

/* CommonPrefixWith() */