[Bug 52704] New: Open File dialog doesn't work on FreeHand 9 demo

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Bug ID: 52704
           Summary: Open File dialog doesn't work on FreeHand 9 demo
           Product: Wine
           Version: 7.4
          Hardware: x86-64
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Created attachment 72061
  --> https://bugs.winehq.org/attachment.cgi?id=72061
FreeHand9 log

After installing the demo, when you click on the "Open" icon on the "Welcome to
FreeHand 9" wizard window, nothing happens visually but the
err:module:DelayLoadFailureHook message appears in the log.

This bug looks very similar to bug 45938 which AF analyzed.  I'm not sure what
was meant by "try GetModuleHandleExA() first before LoadLibraryA()" so I wasn't
able to verify that it resolves this bug.

[Bug 52704] Open File dialog doesn't work on FreeHand 9 demo

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Drew Bugs  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://bugs.winehq.org/sho
                   |                            |w_bug.cgi?id=45936
                URL|                            |https://web.archive.org/web
                   |                            |/20040607015226if_/http://d
                   |                            |ownload.macromedia.com:80/p
                   |                            |ub/freehand/esd/freehand9-t
                   |                            |rial.exe
       Distribution|---                         |Ubuntu

[Bug 52704] FreeHand 9 demo needs unimplemented uxtheme.dll.OpenThemeDataForDpi

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Fabian Maurer  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
          Component|-unknown                    |uxtheme
     Ever confirmed|0                           |1
                 CC|                            |[e-mail]
            Summary|Open File dialog doesn't    |FreeHand 9 demo needs
                   |work on FreeHand 9 demo     |unimplemented
                   |                            |uxtheme.dll.OpenThemeDataFo
                   |                            |rDpi
             Status|UNCONFIRMED                 |NEW

--- Comment #1 from Fabian Maurer  ---
Confirming.

[Bug 52704] FreeHand 9 demo: failure to delay load uxtheme.dll.OpenThemeDataForDpi

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|FreeHand 9 demo needs       |FreeHand 9 demo: failure to
                   |unimplemented               |delay load
                   |uxtheme.dll.OpenThemeDataFo |uxtheme.dll.OpenThemeDataFo
                   |rDpi                        |rDpi

--- Comment #2 from Dmitry Timoshkov  ---
uxtheme.OpenThemeDataForDpi is implemented. Similar problem was analyzed by AF
in the bug 45936 (45938 in the original comment is probably a typo).

[Bug 52704] FreeHand 9 demo: failure to delay load uxtheme.dll.OpenThemeDataForDpi

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

--- Comment #3 from Nikolay Sivov  ---
Created attachment 72264
  --> https://bugs.winehq.org/attachment.cgi?id=72264
patch

This is not related to delayed import. The problem is an exception raised when
uxtheme is attached - registry handle is closed twice, and when being debugged
this raises an exception from NtClose(). No idea if debugging itself is normal
for this application, but attached patch works for me.

[Bug 52704] FreeHand 9 demo: invalid handle exception (0xc0000008) when attaching uxtheme.dll

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|FreeHand 9 demo: failure to |FreeHand 9 demo: invalid
                   |delay load                  |handle exception
                   |uxtheme.dll.OpenThemeDataFo |(0xc0000008) when attaching
                   |rDpi                        |uxtheme.dll

[Bug 52704] FreeHand 9 demo: invalid handle exception (0xc0000008) when attaching uxtheme.dll

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch

--- Comment #4 from Dmitry Timoshkov  ---
(In reply to Nikolay Sivov from comment #3)
> Created attachment 72264 [details]
> patch
>
> This is not related to delayed import. The problem is an exception raised
> when uxtheme is attached - registry handle is closed twice, and when being
> debugged this raises an exception from NtClose(). No idea if debugging
> itself is normal for this application, but attached patch works for me.

The patch works, thanks Nikolay.

Freehand demo runs under built-in debugger, probably to prevent cracking,
see https://bugs.winehq.org/show_bug.cgi?id=37609 for details.

[Bug 52704] FreeHand 9 demo: invalid handle exception (0xc0000008) when attaching uxtheme.dll

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEW                         |RESOLVED
      Fixed by SHA1|                            |28d572aa14300b7452f491e2a6b
                   |                            |53b4e1ad9a104

--- Comment #5 from Nikolay Sivov  ---
Marking fixed, 28d572aa14300b7452f491e2a6b53b4e1ad9a104.

[Bug 52704] FreeHand 9 demo: invalid handle exception (0xc0000008) when attaching uxtheme.dll

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Alexandre Julliard  changed:

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

--- Comment #6 from Alexandre Julliard  ---
Closing bugs fixed in 7.8.

[Bug 52704] FreeHand 9 demo: invalid handle exception (0xc0000008) when attaching uxtheme.dll

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Michael Stefaniuc  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |7.0.x

[Bug 52704] FreeHand 9 demo: invalid handle exception (0xc0000008) when attaching uxtheme.dll

[WineHQ Bugzilla]

https://bugs.winehq.org/show_bug.cgi?id=52704

Michael Stefaniuc  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|7.0.x                       |---

--- Comment #7 from Michael Stefaniuc  ---
Removing the 7.0.x milestone from bug fixes included in 7.0.1.