Nikolay Sivov : ole32/composite: Disable cloning of enumerator instances.
Alexandre Julliard
julliard at winehq.org
Thu Sep 30 16:04:42 CDT 2021
Module: wine
Branch: master
Commit: b557c395556a9d2a6b62c7be3b79968e9dc78e80
URL: https://source.winehq.org/git/wine.git/?a=commit;h=b557c395556a9d2a6b62c7be3b79968e9dc78e80
Author: Nikolay Sivov <nsivov at codeweavers.com>
Date: Thu Sep 30 11:26:47 2021 +0300
ole32/composite: Disable cloning of enumerator instances.
Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>
---
dlls/ole32/compositemoniker.c | 6 +++---
dlls/ole32/tests/moniker.c | 12 ++++--------
2 files changed, 7 insertions(+), 11 deletions(-)
diff --git a/dlls/ole32/compositemoniker.c b/dlls/ole32/compositemoniker.c
index cde3f1762f0..652962e1a87 100644
--- a/dlls/ole32/compositemoniker.c
+++ b/dlls/ole32/compositemoniker.c
@@ -1363,14 +1363,14 @@ EnumMonikerImpl_Reset(IEnumMoniker* iface)
static HRESULT WINAPI EnumMonikerImpl_Clone(IEnumMoniker *iface, IEnumMoniker **ret)
{
- EnumMonikerImpl *This = impl_from_IEnumMoniker(iface);
-
TRACE("%p, %p.\n", iface, ret);
if (!ret)
return E_INVALIDARG;
- return EnumMonikerImpl_CreateEnumMoniker(This->tabMoniker,This->tabSize,This->currentPos,TRUE,ret);
+ *ret = NULL;
+
+ return E_NOTIMPL;
}
static const IEnumMonikerVtbl VT_EnumMonikerImpl =
diff --git a/dlls/ole32/tests/moniker.c b/dlls/ole32/tests/moniker.c
index a60472951f7..abd17c9961f 100644
--- a/dlls/ole32/tests/moniker.c
+++ b/dlls/ole32/tests/moniker.c
@@ -3699,10 +3699,9 @@ todo_wine {
enummoniker2 = (void *)0xdeadbeef;
hr = IEnumMoniker_Clone(enummoniker, &enummoniker2);
-todo_wine {
ok(hr == E_NOTIMPL, "Unexpected hr %#x.\n", hr);
ok(!enummoniker2, "Unexpected pointer.\n");
-}
+
hr = IEnumMoniker_Reset(enummoniker);
ok(hr == S_OK, "Unexpected hr %#x.\n", hr);
@@ -3718,10 +3717,9 @@ todo_wine {
enummoniker2 = (void *)0xdeadbeef;
hr = IEnumMoniker_Clone(enummoniker, &enummoniker2);
-todo_wine {
ok(hr == E_NOTIMPL, "Unexpected hr %#x.\n", hr);
ok(!enummoniker2, "Unexpected pointer.\n");
-}
+
IEnumMoniker_Release(enummoniker);
/* Backward direction */
@@ -3783,10 +3781,9 @@ todo_wine {
enummoniker2 = (void *)0xdeadbeef;
hr = IEnumMoniker_Clone(enummoniker, &enummoniker2);
-todo_wine {
ok(hr == E_NOTIMPL, "Unexpected hr %#x.\n", hr);
ok(!enummoniker2, "Unexpected pointer.\n");
-}
+
hr = IEnumMoniker_Reset(enummoniker);
ok(hr == S_OK, "Unexpected hr %#x.\n", hr);
@@ -3802,10 +3799,9 @@ todo_wine {
enummoniker2 = (void *)0xdeadbeef;
hr = IEnumMoniker_Clone(enummoniker, &enummoniker2);
-todo_wine {
ok(hr == E_NOTIMPL, "Unexpected hr %#x.\n", hr);
ok(!enummoniker2, "Unexpected pointer.\n");
-}
+
IEnumMoniker_Release(enummoniker);
IMoniker_Release(moniker);
More information about the wine-cvs
mailing list