[Bug 21210] New: Clicking on URL hyperlinks in PowerPoint 2007 results in a crash

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

Summary: Clicking on URL hyperlinks in PowerPoint 2007 results
                    in a crash
           Product: Wine
           Version: 1.1.35
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=25474)
 --> (http://bugs.winehq.org/attachment.cgi?id=25474)
PowerPoint 2007 terminal output

As a follow-up to bug 20144, I noticed that with wine-1.1.35-364-g9dc584d,
clicking on an embedded URL hyperlink in a presentation, such as
http://winehq.org/, results in a crash dialog stating:

"Microsoft Office PowerPoint has encountered a problem and needs to close. We
are sorry for the inconvenience."

The complete output is attached.

[Bug 21210] Clicking on URL hyperlinks in PowerPoint 2007 results in a crash

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

Andrew Eikum  changed:

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

--- Comment #1 from Andrew Eikum   2010-01-07 14:41:30 ---
The crash is caused by the wrong codepath being executed.  IHlink_fnNavigate
binds its target to an object and navigates to it.  Since we have an mshtml
implementation, the URLMoniker is bound as an HTMLDocument object.  This
HTMLDocument object is then navigated to, but the HTMLDocument has no client on
which to render (or something like that).  The null pointer causes a crash in
mshtml's OleObject_DoVerb.

Instead, IHlink_fnNavigate should be passing the URL off to ShellExecute.  This
requires the bind to fail, which it would if we didn't try to use mshtml.  The
solution to this problem, I think, is to have some sort of identifier in
IHlink, providing a way to determine whether the IHlink object was created by
mshtml (in which case we want to continue with the bind as we do currently) or
by an application (in which case we want to do whatever native does when a link
is being navigated to -- that is, use ShellExecute).

I've asked Jacek Caban to give his opinion.  Problem identified, solution
(hopefully) incoming.

[Bug 21210] Clicking on URL hyperlinks in PowerPoint 2007 results in a crash

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

--- Comment #2 from Rosanne DiMesio  2011-01-08 17:21:29 CST ---
Seems to be fixed in 1.3.11.

[Bug 21210] Clicking on URL hyperlinks in PowerPoint 2007 results in a crash

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

Rosanne DiMesio  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #3 from Rosanne DiMesio  2011-02-20 22:13:08 CST ---
Marking fixed.

[Bug 21210] Clicking on URL hyperlinks in PowerPoint 2007 results in a crash

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

Alexandre Julliard  changed:

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

--- Comment #4 from Alexandre Julliard  2011-03-04 12:35:35 CST ---
Closing bugs fixed in 1.3.15.