[Bug 9392] New: Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

Summary: Recent changes to propsheet.c causes crashes in
                    MagnaCura
           Product: Wine
           Version: unspecified
          Platform: Other
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: wine-comctl32
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

This patch to propsheet.c causes Wine to crash when using the application Magna
Cura Äho:

+  /* unset active page while doing this transition. */
+  if (psInfo->active_page != -1)
+     ShowWindow(psInfo->proppage[psInfo->active_page].hwndPage, SW_HIDE);
+  psInfo->active_page = -1;

It was added in revision 1.169 of propsheet.c. Removing these lines avoids the
crash. The other parts of the changes done in revision 1.169 seems ok.

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

--- Comment #1 from Peter Åstrand   2007-08-20 10:47:58 ---
Created an attachment (id=7691)
 --> (http://bugs.winehq.org/attachment.cgi?id=7691)
Backtrace and log with WINDEBUG=+propsheet, using rev 1.175

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail],
                   |                            |[e-mail]
                URL|                            |http://www.pulsen.se/

--- Comment #2 from Dan Kegel   2007-08-20 11:37:20 ---
That was Aric's patch, ccing him.

I don't think the app has a downloadable trial, but I'm adding
a URL for the vendor.

BTW, since you're working with the crossover code
tree, you should be talking with Codeweavers, not winehq.
Next time you report a bug to winehq. please verify
it and provide a log using the latest winehq git tree.
Thanks!

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

--- Comment #3 from Peter Åstrand   2007-08-20 11:57:25 ---
> I don't think the app has a downloadable trial, but I'm adding
> a URL for the vendor.

Correct. And even if you have the application, you'll need the commercial
Pervasive Client as well as a SQL server.
> BTW, since you're working with the crossover code
> tree, you should be talking with Codeweavers, not winehq.
> Next time you report a bug to winehq. please verify
> it and provide a log using the latest winehq git tree.
> Thanks!

I knew this one was coming :) Actually, Alexandre Julliard recommended me to
submit a bug here.

I can verify things with complete, unmodified C files from the Wine project
within CrossOver, but testing the application in a pristine Wine is just too
much work for me. Besides, it wouldn't even install: It requires Internet
Explorer, which (still?) doesn't work in plain Wine.

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

--- Comment #4 from Alexandre Julliard   2007-08-21 05:34:54 ---
It looks like the app destroys the prop sheet if it doesn't find an active page
while creating a new one. Probably some sort of message order issue, this will
require writing some test cases.

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

--- Comment #5 from Austin English   2008-04-14 23:46:30 ---
Is this still an issue in current (0.9.59 or newer) wine?

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

--- Comment #6 from Peter Åstrand   2008-04-15 02:36:31 ---
> Is this still an issue in current (0.9.59 or newer) wine?

I cannot confirm that; this project is currently halted.

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

--- Comment #7 from Austin English   2008-10-22 13:55:54 ---
Abandoned then. Please reopen if you can test again in current wine and still
have a problem.

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

James Hawkins  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |ABANDONED

--- Comment #8 from James Hawkins   2008-10-22 14:14:12 ---
Actually marking abandoned.

[Bug 9392] Recent changes to propsheet.c causes crashes in MagnaCura

http://bugs.winehq.org/show_bug.cgi?id=9392

James Hawkins  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #9 from James Hawkins   2008-10-22 14:14:21 ---
Closing.