https://bugs.winehq.org/show_bug.cgi?id=48732
--- Comment #9 from Gabriel Ivăncescu gabrielopcode@gmail.com --- (In reply to Zebediah Figura from comment #8)
(In reply to Gabriel Ivăncescu from comment #6)
Right, I was thinking the app does something weird like hook into SetParent. Or perhaps the call exposes a race condition of a deeper issue, which is a bigger concern. (this would just "hide" it)
That seems very unlikely, but if the patch really helps, I don't even know what to think. I'd really like to figure out *why* the patch helps before accepting it, honestly.
Maybe it's due to SetWindowLongPtr rather than SetParent itself. Possible causes I can think of that might be related:
* SetWindowLongPtr does a wineserver call. * It sends WM_STYLECHANGING message. * It calls a winex11 driver function, X11DRV_SetWindowStyle.
I'm speculating at this point, because I can't reproduce the crash either. But I suspect the last one. It probably has to do with a specific Window Manager or similar (perhaps that's why it also applies only to full-screen).
Regardless, should I send it to the mailing list for upstream inclusion? Since it does help with the crash.