[Bug 54531] New: jsproxy:jsproxy crashes on Windows 11

[WineHQ Bugzilla]

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

Bug ID: 54531
           Summary: jsproxy:jsproxy crashes on Windows 11
           Product: Wine
           Version: unspecified
          Hardware: x86-64
                OS: Windows
            Status: NEW
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: [e-mail]
          Reporter: [e-mail]

jsproxy:jsproxy crashes on Windows 11:

jsproxy.c:52: Test failed: got 3735928559
jsproxy.c:56: Test failed: got 3735928559
jsproxy.c:59: Test failed: got 3735928559
jsproxy.c:60: this is the last test seen before the exception
054c:jsproxy: unhandled exception c0000005 at 004013FA

See https://test.winehq.org/data/patterns.html#jsproxy:jsproxy

Where 3735928559 == 0xdeadbeef

So InternetGetProxyInfo() fails but does not set LastError. Furthermore the
proxy pointer is not initialized either so the test crashes when trying to free
it.

[Bug 54531] jsproxy:jsproxy crashes on Windows 11

[WineHQ Bugzilla]

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

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |source, testcase

--- Comment #1 from François Gouget  ---
Microsoft indicates that wininet.InternetGetProxyInfo() is deprecated on
Windows 10 and not supported on Windows 11.

This test calls the jsproxy implementation instead and I don't know if that's
built on top of wininet or the reverse. But it seems like that implementation
is not supported all the same on Windows 11 (we don't have tests of the wininet
implementation that could confirm that).

Although Microsoft's documentation does not say so, it seems like the same is
true for jsproxy.InternetInitializeAutoProxyDll().

So I sent a merge request to just skip the tests when the function fails and
does not set LastError.
https://gitlab.winehq.org/wine/wine/-/merge_requests/2233

[Bug 54531] jsproxy:jsproxy crashes on Windows 11

[WineHQ Bugzilla]

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

François Gouget  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |00b6ce35bf2739ad6c66dd47b41
                   |                            |ec754d0387aaa
             Status|NEW                         |RESOLVED

--- Comment #2 from François Gouget  ---
This is fixed:

commit 00b6ce35bf2739ad6c66dd47b41ec754d0387aaa
Author:     François Gouget 
AuthorDate: Sun Feb 19 19:32:18 2023 +0100

jsproxy/tests: InternetInitializeAutoProxyDll() is no longer supported on
Windows 11.

The same goes for InternetGetProxyInfo(). They seem to have been
    replaced with stubs that fail but don't bother setting the error code.
    So just skip the tests when that happens.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=54531

[Bug 54531] jsproxy:jsproxy crashes on Windows 11

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

--- Comment #3 from Alexandre Julliard  ---
Closing bugs fixed in 8.4.