gdi32: Do not sign extend 64-bit gdi handles.

Testing on Windows shows typically handles are not sign extended, however sometimes they are. It's not clear what triggers the change in behaviour. Signed-off-by: Francisco Casas <fcasas@codeweavers.com> Signed-off-by: Huw Davies <huw@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

gdi32: Do not sign extend 64-bit gdi handles.
379333e3 · Francisco Casas · Alexandre Julliard · d85b700d · 379333e3 · 379333e3
Commit 379333e3 authored Oct 27, 2021 by Francisco Casas Committed by Alexandre Julliard Oct 27, 2021
Show whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

objects.c dlls/gdi32/objects.c +1 -1

gdiobj.c dlls/win32u/gdiobj.c +1 -1

No files found.
--- a/dlls/gdi32/objects.c
+++ b/dlls/gdi32/objects.c
@@ -102,7 +102,7 @@ static inline GDI_HANDLE_ENTRY *handle_entry( HGDIOBJ handle )
 static HGDIOBJ entry_to_handle( GDI_HANDLE_ENTRY *entry )
 {
    unsigned int idx = entry - get_gdi_shared()->Handles;
-    return LongToHandle( idx | (entry->Unique << NTGDI_HANDLE_TYPE_SHIFT) );
+    return ULongToHandle( idx | (entry->Unique << NTGDI_HANDLE_TYPE_SHIFT) );
 }

 static DWORD get_object_type( HGDIOBJ obj )

--- a/dlls/win32u/gdiobj.c
+++ b/dlls/win32u/gdiobj.c
@@ -55,7 +55,7 @@ const struct user_callbacks *user_callbacks = NULL;
 static inline HGDIOBJ entry_to_handle( GDI_HANDLE_ENTRY *entry )
 {
    unsigned int idx = entry - gdi_shared->Handles;
-    return LongToHandle( idx | (entry->Unique << NTGDI_HANDLE_TYPE_SHIFT) );
+    return ULongToHandle( idx | (entry->Unique << NTGDI_HANDLE_TYPE_SHIFT) );
 }

 static inline GDI_HANDLE_ENTRY *handle_entry( HGDIOBJ handle )