[Bug 51899] 16-bit Collins Dictionary has black background.
WineHQ Bugzilla
wine-bugs at winehq.org
Fri Oct 22 06:51:16 CDT 2021
https://bugs.winehq.org/show_bug.cgi?id=51899
--- Comment #6 from Bruni <earns.61 at gmail.com> ---
My bad
According to PTHREAD specification, it does not guarantee that instructions
inside the lock itself are protected from reordering.
So, it's better to put multiple locks.
DC_ATTR *get_dc_attr( HDC hdc )
{
pthread_mutex_t lock = PTHREAD_MUTEX_INITIALIZER;
pthread_mutex_lock( &lock );
DWORD type = gdi_handle_type( hdc );
DC_ATTR *dc_attr;
pthread_mutex_unlock( &lock );
pthread_mutex_lock( &lock );
unsigned int comparedType = type & 0x1f0000;
pthread_mutex_unlock( &lock );
pthread_mutex_lock( &lock );
unsigned int ntgdiObjDc = comparedType & NTGDI_OBJ_DC;
pthread_mutex_unlock( &lock );
pthread_mutex_lock( &lock );
dc_attr = get_gdi_client_ptr( hdc, 0 );
pthread_mutex_unlock( &lock );
pthread_mutex_lock( &lock );
if (!ntgdiObjDc)
{
SetLastError( ERROR_INVALID_HANDLE );
return NULL;
}
pthread_mutex_unlock( &lock );
pthread_mutex_lock( &lock );
if (!dc_attr)
{
SetLastError( ERROR_INVALID_HANDLE );
return NULL;
}
pthread_mutex_unlock( &lock );
return dc_attr->disabled ? NULL : dc_attr;
}
--
Do not reply to this email, post in Bugzilla using the
above URL to reply.
You are receiving this mail because:
You are watching all bug changes.
More information about the wine-bugs
mailing list