Commit b7bf7f01 authored by Alexandre Julliard's avatar Alexandre Julliard

user32: Avoid releasing a potentially null pointer.

parent bdfab8cd
...@@ -180,7 +180,7 @@ void *free_user_handle( HANDLE handle, enum user_obj_type type ) ...@@ -180,7 +180,7 @@ void *free_user_handle( HANDLE handle, enum user_obj_type type )
else InterlockedCompareExchangePointer( &user_handles[index], NULL, ptr ); else InterlockedCompareExchangePointer( &user_handles[index], NULL, ptr );
} }
SERVER_END_REQ; SERVER_END_REQ;
release_user_handle_ptr( ptr ); USER_Unlock();
} }
return ptr; return ptr;
} }
...@@ -290,7 +290,7 @@ static void free_window_handle( HWND hwnd ) ...@@ -290,7 +290,7 @@ static void free_window_handle( HWND hwnd )
else InterlockedCompareExchangePointer( &user_handles[index], NULL, ptr ); else InterlockedCompareExchangePointer( &user_handles[index], NULL, ptr );
} }
SERVER_END_REQ; SERVER_END_REQ;
release_user_handle_ptr( ptr ); USER_Unlock();
HeapFree( GetProcessHeap(), 0, ptr ); HeapFree( GetProcessHeap(), 0, ptr );
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment