[Bug 23591] New: EditPadPro: Access violation when using function navigator with mouse double-click
http://bugs.winehq.org/show_bug.cgi?id=23591
Summary: EditPadPro: Access violation when using function navigator with mouse double-click Product: Wine Version: 1.2-rc6 Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: minor Priority: P2 Component: -unknown AssignedTo: wine-bugs@winehq.org ReportedBy: jarekczek@poczta.onet.pl
Created an attachment (id=29490) --> (http://bugs.winehq.org/attachment.cgi?id=29490) last 100 lines of +relay
Open the source code file, eg. /usr/src/wine-1.2-rc6/dlls/user32/winproc.c Open the File Navigator window with Ctrl-F9 (left side pane) Try double-clicking a function name in this pane. You get: Access violation at address 00559FCA in module 'EditPadPro.exe'. Read of address 00000018. After closing this message you can go on. The function we called positions the editor at the right place and places focus in the right pane. It works as expected. The only thing is this message.
This may be connected with windows switching / activating / deactivating as it is a 2 pane app.
Prerequisites: After installing the demo app http://download.jgsoft.com/editpad/SetupEditPadProDemo.exe follow the guidelines from AppDb http://appdb.winehq.org/objectManager.php?sClass=version&iId=20710 to properly configure it for wine. Just 2 simple steps.
Additional info: The same result as double-clicking can be achieved by clicking once and following it with the enter key. This does not cause any problems. Trying to locate the same function name many times doesn't bring the access violation banner any more. It appears only when you try to follow a new function. After that the function is freed from the annoying message box.
I contacted Jan Goyvaerts, the author of the app. He is familiar with wine and he says this is a wine bug.
What else can I provide to make it solved? Brothers programmers, this editor is a powerful crossplatform tool for editing code. It's worth investigating some time. I spent already several hours to test this bug and report it. Maybe I should make another report with specific WINEDEBUG options?
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #1 from Jarek jarekczek@poczta.onet.pl 2010-07-10 03:36:08 --- Created an attachment (id=29491) --> (http://bugs.winehq.org/attachment.cgi?id=29491) +loaddll
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #2 from Jeff Zaroyko jeffz@jeffz.name 2010-07-10 04:05:15 --- attach regular terminal output without WINEDEBUG options, if no backtrace is printed then the application is probably obscuring it, you can try getting one using winedbg.
http://wiki.winehq.org/Backtraces
http://bugs.winehq.org/show_bug.cgi?id=23591
Louis Lenders xerox_xerox2000@yahoo.co.uk changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |download Status|UNCONFIRMED |NEW URL| |http://download.jgsoft.com/ | |editpad/SetupEditPadProDemo | |.exe CC| |xerox_xerox2000@yahoo.co.uk Component|-unknown |comctl32 Ever Confirmed|0 |1
--- Comment #3 from Louis Lenders xerox_xerox2000@yahoo.co.uk 2010-07-10 04:42:23 --- Confirming. Looks like a bug in comctl32
I tried 'winetricks cc580' but that gave a whole series of access violations at startup (maybe that comctl32 is too old?), but the following worked for me (after creating a new clean ~/.wine):
1.) rm -rf ~/.wine/drive_c/windows/winsxs/manifests/x86_microsoft.windows.common-controls_6595b64144ccf1df_6.0.2600.2982_none_deadbeef.manifest
2.) copy a comctl32.dll from my Windows XP into ~/.wine/drive_c/windows/system32/
3. Install EditPadProDemo.exe and run the app by : WINEDLLOVERRIDES="comctl32=n" wine EditPadPro.exe
No AccessViolations anymore, so most likely a bug in wine's comctl32
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #4 from Jarek jarekczek@poczta.onet.pl 2010-07-10 06:00:16 --- Thank you guys, you're really fast! Thanks for the hint about winedbg and backtraces. I'll try this. Now thanks to the deadbeef solution I have no violations. The bug was the same on 1.2.rc7.
http://bugs.winehq.org/show_bug.cgi?id=23591
Vitaliy Margolen vitaliy@kievinfo.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Severity|minor |normal
http://bugs.winehq.org/show_bug.cgi?id=23591
Jarek jarekczek@poczta.onet.pl changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jarekczek@poczta.onet.pl
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #5 from Nikolay Sivov bunglehead@gmail.com 2010-07-11 02:11:01 --- Please try to figure out what control component is affected. If plain output without switches doesn't have anything related attach complete +relay log.
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #6 from Andrew Nguyen arethusa26@gmail.com 2010-07-11 07:04:03 --- Created an attachment (id=29523) --> (http://bugs.winehq.org/attachment.cgi?id=29523) EditPadPro dialog backtrace
The backtrace of the access violation dialog implies a treeview problem. There wasn't any plain output, so I'll attach the dialog backtrace and compressed relay log.
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #7 from Andrew Nguyen arethusa26@gmail.com 2010-07-11 07:06:06 --- Created an attachment (id=29524) --> (http://bugs.winehq.org/attachment.cgi?id=29524) EditPadPro +relay,+seh,+tid trace (lzma compressed)
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #8 from Nikolay Sivov bunglehead@gmail.com 2011-05-14 09:07:00 CDT --- Seems to me it's a problem with wrong expand/collapse logic. When item has no children we shouldn't send TVN_ITEMEXPANDING but we do.
I'll take a closer look.
http://bugs.winehq.org/show_bug.cgi?id=23591
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|wine-bugs@winehq.org |bunglehead@gmail.com
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #9 from Nikolay Sivov bunglehead@gmail.com 2011-05-15 15:21:32 CDT --- Patch sent http://www.winehq.org/pipermail/wine-patches/2011-May/101934.html
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #10 from Jarek jarekczek@poczta.onet.pl 2011-05-16 10:51:12 CDT --- Nikolay, nice to hear you fixed that. I would give it a try, but can't apply your patch to morning's git. Where you have:
- if (item->state & TVIS_EXPANDED) - return TRUE;
I have:
if (wineItem->state & TVIS_EXPANDED) return TRUE;
(treeview.c, TREEVIEW_Expand, line 3334)
I still use this app under wine, almost every day, so I could test it.
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #11 from Nikolay Sivov bunglehead@gmail.com 2011-05-28 15:03:43 CDT --- This patch breaks some tests so it won't be committed as it is. I fixed recently toggle mode that sent unneeded notifications too, this one still needs to be fixed.
http://bugs.winehq.org/show_bug.cgi?id=23591
Daniel Jelinski djelinski1@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |djelinski1@gmail.com
--- Comment #12 from Daniel Jelinski djelinski1@gmail.com 2012-10-16 23:25:49 CDT --- TVN_ITEMEXPANDING has been fixed. Please retest in gurrent git.
http://bugs.winehq.org/show_bug.cgi?id=23591
Nikolay Sivov bunglehead@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|bunglehead@gmail.com |wine-bugs@winehq.org
http://bugs.winehq.org/show_bug.cgi?id=23591
--- Comment #13 from Jarek jarekczek@poczta.onet.pl 2012-10-17 15:52:30 CDT --- I confirm that today's git is free of the error. I don't use the editor anymore so I only did a test on one file. I switched to crossplatform java based jEdit.
http://bugs.winehq.org/show_bug.cgi?id=23591
Austin English austinenglish@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |RESOLVED Resolution| |FIXED
--- Comment #14 from Austin English austinenglish@gmail.com 2012-10-17 18:18:58 CDT --- (In reply to comment #13)
I confirm that today's git is free of the error. I don't use the editor anymore so I only did a test on one file. I switched to crossplatform java based jEdit.
Thanks.
http://bugs.winehq.org/show_bug.cgi?id=23591
Alexandre Julliard julliard@winehq.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED
--- Comment #15 from Alexandre Julliard julliard@winehq.org 2012-10-26 13:29:27 CDT --- Closing bugs fixed in 1.5.16.
-
wine-bugs＠winehq.org