https://bugs.winehq.org/show_bug.cgi?id=7425 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Keywords| |patch --- Comment #35 from Bruno Jesus <00cpxxx(a)gmail.com> --- (In reply to Marc Bessières from comment #34)
Thank you Bruno for the fix. I'm glad you found the issue obvious as it wasn't for me :-)
Your backtrace is certainly cryptic to me too, I was talking about my backtrace with wine compiled without optimization (-O0). This make it much more clear and with better source information.
Thanks Béla for the testing, as you I can confirm that this fixes the crash.
Yes, thanks Béla once again =)
When I wanted to make the change of your patch manually I found out that the exact same function exists in 3 places. So shouldn't they all be corrected? See attachment for such a patch.
You are right, the three of them need to be fixed but I'm not sure a single patch is correct because it touches 3 different DLLs, I'll try sending upstream to get some feedback. By the way, before the chunk of changes in a patch there is always the full path to the file. -- 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.