[PATCH 0/1] MR10924: win32u: Set Owner.ProcessId in GDI shared handle table entries

[Bartosz Kosiorek (＠gang65)]

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

[PATCH 1/1] win32u: Set Owner.ProcessId in GDI shared handle table entries

[Bartosz Kosiorek]

From: Bartosz Kosiorek <[e-mail]>

---
 dlls/gdi32/tests/gdiobj.c | 2 --
 dlls/win32u/gdiobj.c      | 1 +
 2 files changed, 1 insertion(+), 2 deletions(-)

diff --git a/dlls/gdi32/tests/gdiobj.c b/dlls/gdi32/tests/gdiobj.c
index ffe376abed7..6d0c636910b 100644
--- a/dlls/gdi32/tests/gdiobj.c
+++ b/dlls/gdi32/tests/gdiobj.c
@@ -430,7 +430,6 @@ static void test_shared_handle_table(void)
     ok(hrgn != 0, "CreateRectRgn failed\n");
     handle = HandleToULong( hrgn );
     entry = &gdi_shared->Handles[handle & 0xffff];
-    todo_wine
     ok(entry->Owner.ProcessId == GetCurrentProcessId(), "ProcessId = %x, expected %lx\n",
        entry->Owner.ProcessId, GetCurrentProcessId());
 
@@ -442,7 +441,6 @@ static void test_shared_handle_table(void)
     todo_wine
     ok(entry->Type == 4, "Type = %x\n", entry->Type);
     ok(entry->Object, "Object = NULL\n");
-    todo_wine
     ok(entry->Owner.ProcessId == GetCurrentProcessId(), "ProcessId = %x, expected %lx\n",
        entry->Owner.ProcessId, GetCurrentProcessId());
     ok(entry->Owner.Count == 0, "Count = %u\n", entry->Owner.Count);
diff --git a/dlls/win32u/gdiobj.c b/dlls/win32u/gdiobj.c
index e861dee167d..29681023d02 100644
--- a/dlls/win32u/gdiobj.c
+++ b/dlls/win32u/gdiobj.c
@@ -742,6 +742,7 @@ HGDIOBJ alloc_gdi_handle( struct gdi_obj_header *obj, DWORD type, const struct g
     entry->Object  = (UINT_PTR)obj;
     entry->ExtType = type >> NTGDI_HANDLE_TYPE_SHIFT;
     entry->Type    = entry->ExtType & 0x1f;
+    entry->Owner.ProcessId = GetCurrentProcessId();
     if (++entry->Generation == 0x80) entry->Generation = 1;
     ret = entry_to_handle( entry );
     pthread_mutex_unlock( &gdi_lock );
-- 
GitLab

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

[Jinoh Kang (＠iamahuman)]

Thanks for the contribution! Which application needs this?

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/10924#note_141333