[PATCH v2 0/1] MR7781: combase: Add stub for RoOriginateErrorW.

[Louis Lenders (＠xe)]

This is called by UniGetUi (https://github.com/marticliment/UniGetUI/releases/tag/3.1.8).

(After choosing 'Install for all users' --> 'perform portable installation' otherwise the installer runs into other (shellfolder) related bugs)

--
  v2: combase: Add stub for RoOriginateErrorW.
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7781

[PATCH v2 1/1] combase: Add stub for RoOriginateErrorW.

[Louis Lenders]

From: Louis Lenders <xerox.xerox2000x(a)gmail.com>

This is called by UniGetUi (https://github.com/marticliment/UniGetUI/releases/tag/3.1.8).

(After choosing 'Install for all users' --> 'perform portable installation' otherwise the installer runs into other (shellfolder) related bugs)
---
 dlls/combase/combase.spec | 2 +-
 dlls/combase/roapi.c      | 9 +++++++++
 include/roerrorapi.h      | 1 +
 3 files changed, 11 insertions(+), 1 deletion(-)

diff --git a/dlls/combase/combase.spec b/dlls/combase/combase.spec
index ae13be7e6cc..d8f1fe83653 100644
--- a/dlls/combase/combase.spec
+++ b/dlls/combase/combase.spec
@@ -302,7 +302,7 @@
 @ stub RoInspectCapturedStackBackTrace
 @ stub RoInspectThreadErrorInfo
 @ stdcall RoOriginateError(long ptr)
-@ stub RoOriginateErrorW
+@ stdcall RoOriginateErrorW(long long wstr)
 @ stdcall RoOriginateLanguageException(long ptr ptr)
 @ stub RoParameterizedTypeExtraGetTypeSignature
 @ stdcall RoRegisterActivationFactories(ptr ptr long ptr)
diff --git a/dlls/combase/roapi.c b/dlls/combase/roapi.c
index 8118758d338..39e7dce63eb 100644
--- a/dlls/combase/roapi.c
+++ b/dlls/combase/roapi.c
@@ -507,6 +507,15 @@ BOOL WINAPI RoOriginateError(HRESULT error, HSTRING message)
     return FALSE;
 }
 
+/***********************************************************************
+ *      RoOriginateErrorW (combase.@)
+ */
+BOOL WINAPI RoOriginateErrorW(HRESULT error, UINT max_len, const WCHAR *message)
+{
+    FIXME("%#lx, %u, %s: stub\n", error, max_len, debugstr_w(message));
+    return FALSE;
+}
+
 /***********************************************************************
  *      RoSetErrorReportingFlags (combase.@)
  */
diff --git a/include/roerrorapi.h b/include/roerrorapi.h
index 21fe4eb09dc..4831e3afe40 100644
--- a/include/roerrorapi.h
+++ b/include/roerrorapi.h
@@ -34,6 +34,7 @@ typedef enum
 
 HRESULT WINAPI GetRestrictedErrorInfo(IRestrictedErrorInfo **info);
 BOOL    WINAPI RoOriginateError(HRESULT error, HSTRING message);
+BOOL    WINAPI RoOriginateErrorW(HRESULT error, UINT max_len, const WCHAR *message);
 BOOL    WINAPI RoOriginateLanguageException(HRESULT error, HSTRING message, IUnknown *language_exception);
 HRESULT WINAPI RoSetErrorReportingFlags(UINT32 flags);
 HRESULT WINAPI SetRestrictedErrorInfo(IRestrictedErrorInfo *info);
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/7781

[Huw Davies (＠huw)]

Huw Davies (@huw) commented about dlls/combase/roapi.c:
> return FALSE;
>  }
>
> +/***********************************************************************
> + *      RoOriginateErrorW (combase.@)
> + */
> +BOOL WINAPI RoOriginateErrorW(HRESULT error, UINT max_len, const WCHAR *message)
> +{
> +    FIXME("%#lx, %u, %s: stub\n", error, max_len, debugstr_w(message));

`message` isn't necessarily null-terminated.  You want something like:
```suggestion:-0+0
    FIXME("%#lx, %u, %s: stub\n", error, max_len, debugstr_wn(message, max_len ? max_len : -1));
```

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/7781#note_101690