[Bug 10379] New: unhandled page fault on first attempt to use help viewer

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

Summary: unhandled page fault on first attempt to use help viewer
           Product: Wine
           Version: 0.9.49.
          Platform: PC
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: wine-shdocvw
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

This appears to be a regression from 0.9.48 to 0.9.49.  Symptoms are very
similar to old fixed bug #8077 and not-as-old fixed bug #8851.  An unhandled
page fault happens immediately after installing Wine Gecko.

Application is PAF (Personal Ancestral File) 5.  Application installation
appeared to go well, and basic application functionality appears okay.  Wine
Gecko appears to install, but immediately after installing, or right after
clicking on the yellow question mark "help" icon, an unhandled page fault
happens.

I plan to attach the output with debug channels enabled from bug #8851. 
If/when I can make time to do it, I also plan to run a git regression.

[Bug 10379] unhandled page fault on first attempt to use help viewer

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

--- Comment #1 from Robert Riches   2007-11-10 22:33:17 ---
Created an attachment (id=9092)
 --> (http://bugs.winehq.org/attachment.cgi?id=9092)
Debug log of unhandled page fault.

This is the log with WINEDEBUG set to

+mshtml,+shdocvw,+urlmon,+htmlhelp

To my untrained eye, the line immediately before the unhandled page fault looks
a bit suspicious:

trace:mshtml:nsChannel_AsyncOpen container = NULL

[Bug 10379] unhandled page fault on first attempt to use help viewer

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

Clinton Stimpson  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

--- Comment #2 from Clinton Stimpson   2007-11-17 16:29:53 ---

Regression caused by this commit:

6b213042480602b81303eec17fe5ff0299b51dfe is first bad commit
commit 6b213042480602b81303eec17fe5ff0299b51dfe
Author: Jacek Caban 
Date:   Fri Nov 2 19:53:02 2007 +0100

mshtml: AsyncOpen code clean up.

[Bug 10379] unhandled page fault on first attempt to use help viewer

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

--- Comment #3 from Jacek Caban   2007-11-18 05:31:46 ---
Created an attachment (id=9233)
 --> (http://bugs.winehq.org/attachment.cgi?id=9233)
mshtml: Fixed handling channels without container and necko channel.

Thanks for the regression test. The attached patch should fix it.

[Bug 10379] unhandled page fault on first attempt to use help viewer

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

--- Comment #4 from Robert Riches   2007-11-19 20:06:16 ---
Patch id 9233 appears to have solved this bug as far as I can see.  There is
still the intermittent problem of a blowup after navigating around in the help
window, but that's a different subject.

Thanks!

[Bug 10379] unhandled page fault on first attempt to use help viewer

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

Clinton Stimpson  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #5 from Clinton Stimpson   2007-11-30 18:47:26 ---
0.9.50 is out with this fix.

[Bug 10379] unhandled page fault on first attempt to use help viewer

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

Hans Leidekker  changed:

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

--- Comment #6 from Hans Leidekker   2007-12-01 03:57:05 ---
Confirmed fixed in latest release.