March 23, 2026
8:43 a.m.
Rémi Bernon (@rbernon) commented about dlls/win32u/class.c:
SERVER_END_REQ; if (!ret) { + NtUserDestroyCursor( sm_icon, 0 ); free( class ); + user_unlock(); return 0; }
I'd move `NtUserDestroyCursor` outside of the user lock, as creation happens outside of it as well, and introduce a `goto failed;` here to share the error path with the other error below: ``` failed: user_unlock(); NtUserDestroyCursor( sm_icon, 0 ); free( class ); return 0; ``` -- https://gitlab.winehq.org/wine/wine/-/merge_requests/10271#note_133370