[PATCH 0/1] MR4850: powrprof: Add PowerWriteACValueIndex stub.

[Russell Greene (＠russelltg)]

Add `PowerWriteACValueIndex` stub.

This is required for an internal application to work

It seems like the status-quo is just leaking the outputs of `debugstr_guid`, so I did that....

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

[PATCH 1/1] powrprof: Add PowerWriteACValueIndex stub

[Russell Greene]

From: Russell Greene <russell(a)shotover.com>

---
 dlls/powrprof/powrprof.c    | 6 ++++++
 dlls/powrprof/powrprof.spec | 1 +
 include/powrprof.h          | 1 +
 3 files changed, 8 insertions(+)

diff --git a/dlls/powrprof/powrprof.c b/dlls/powrprof/powrprof.c
index ac1fa34bc05..e1934b12f03 100644
--- a/dlls/powrprof/powrprof.c
+++ b/dlls/powrprof/powrprof.c
@@ -356,6 +356,12 @@ DWORD WINAPI PowerSettingUnregisterNotification(HPOWERNOTIFY handle)
     return ERROR_SUCCESS;
 }
 
+DWORD WINAPI PowerWriteACValueIndex(HKEY RootPowerKey, const GUID *SchemeGuid, const GUID *SubGroupOfPowerSettingsGuid, const GUID *PowerSettingGuid, DWORD AcValueIndex)
+{
+   FIXME("(%p,%s,%s,%s,0x%08lx) stub!\n", RootPowerKey, debugstr_guid(SchemeGuid), debugstr_guid(SubGroupOfPowerSettingsGuid), debugstr_guid(PowerSettingGuid), AcValueIndex);
+   return ERROR_SUCCESS;
+}
+
 BOOL WINAPI DllMain(HINSTANCE hinstDLL, DWORD fdwReason, LPVOID lpvReserved)
 {
    switch(fdwReason) {
diff --git a/dlls/powrprof/powrprof.spec b/dlls/powrprof/powrprof.spec
index 7a65fd37fc2..95d28178729 100644
--- a/dlls/powrprof/powrprof.spec
+++ b/dlls/powrprof/powrprof.spec
@@ -21,6 +21,7 @@
 @ stdcall PowerUnregisterSuspendResumeNotification(ptr)
 @ stdcall PowerSettingRegisterNotification(ptr long ptr ptr)
 @ stdcall PowerSettingUnregisterNotification(ptr)
+@ stdcall PowerWriteACValueIndex(ptr ptr ptr ptr long)
 @ stdcall ReadGlobalPwrPolicy (ptr)
 @ stdcall ReadProcessorPwrScheme (long ptr)
 @ stdcall ReadPwrScheme (long ptr)
diff --git a/include/powrprof.h b/include/powrprof.h
index 4c6cee41d16..28ebf2a7a79 100644
--- a/include/powrprof.h
+++ b/include/powrprof.h
@@ -164,6 +164,7 @@ DWORD   WINAPI PowerRegisterSuspendResumeNotification(DWORD, HANDLE, PHPOWERNOTI
 DWORD   WINAPI PowerUnregisterSuspendResumeNotification(HPOWERNOTIFY);
 DWORD   WINAPI PowerSettingRegisterNotification(const GUID *, DWORD, HANDLE, PHPOWERNOTIFY);
 DWORD   WINAPI PowerSettingUnregisterNotification(HPOWERNOTIFY);
+DWORD   WINAPI PowerWriteACValueIndex(HKEY, const GUID *, const GUID *, const GUID *, DWORD);
 BOOLEAN WINAPI ReadGlobalPwrPolicy(PGLOBAL_POWER_POLICY);
 BOOLEAN WINAPI ReadProcessorPwrScheme(UINT, PMACHINE_PROCESSOR_POWER_POLICY);
 BOOLEAN WINAPI ReadPwrScheme(UINT, PPOWER_POLICY);
-- 
GitLab

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

[Zebediah Figura (＠zfigura)]

> It seems like the status-quo is just leaking the outputs of `debugstr_guid`, so I did that....

Despite the name __wine_dbg_strdup() it doesn't actually leak anything.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4850#note_57493

[Russell Greene (＠russelltg)]

On Tue Jan 16 03:01:09 2024 +0000, Zebediah Figura wrote:
> > It seems like the status-quo is just leaking the outputs of
> > `debugstr_guid`, so I did that....
> > Despite the name __wine_dbg_strdup() it doesn't actually leak anything.
> > Very interesting, neat (and slightly scary) pattern!

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/4850#note_57496