[Bug 49414] New: DefWindowProcW doesn't handle WIN_GetPtr properly when handling WM_GETTEXTLENGTH
https://bugs.winehq.org/show_bug.cgi?id=49414 Bug ID: 49414 Summary: DefWindowProcW doesn't handle WIN_GetPtr properly when handling WM_GETTEXTLENGTH Product: Wine Version: unspecified Hardware: x86 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: user32 Assignee: wine-bugs(a)winehq.org Reporter: gabravier(a)gmail.com Distribution: --- case WM_GETTEXTLENGTH: { WND *wndPtr = WIN_GetPtr( hwnd ); if (wndPtr && wndPtr->text) result = (LRESULT)strlenW(wndPtr->text); WIN_ReleasePtr( wndPtr ); } break; (from dlls/user32/defwnd.c:998) WIN_GetPtr can return WND_OTHER_PROCESS, which this code does not check for (for example, dlls/user32/defwnd.c:550 checks for it properly). A lot of other code in defwnd.c doesn't check for it, so it looks like having a window be valid for another process could cause problems. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=49414 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|UNCONFIRMED |RESOLVED Resolution|--- |INVALID --- Comment #1 from Alexandre Julliard <julliard(a)winehq.org> --- That's already checked at the top of the function. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=49414 --- Comment #2 from Gabriel Ravier <gabravier(a)gmail.com> --- Oh, I'm blind ????. I should have paid more attention to what exactly that code was doing. Welp, sorry for filing a stupid bug report like that ????. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=49414 --- Comment #3 from Austin English <austinenglish(a)gmail.com> --- Closing. -- 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.
https://bugs.winehq.org/show_bug.cgi?id=49414 Austin English <austinenglish(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #4 from Austin English <austinenglish(a)gmail.com> --- Closing. -- 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.
participants (1)
-
WineHQ Bugzilla