[Bug 48297] New: Wineboot crashes on 64bit prefix creation (macOS wine built without mingw)

[WineHQ Bugzilla]

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

Bug ID: 48297
           Summary: Wineboot crashes on 64bit prefix creation (macOS wine
                    built without mingw)
           Product: Wine
           Version: 5.0-rc1
          Hardware: x86-64
                OS: Mac OS X
            Status: NEW
          Keywords: regression, source
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: [e-mail]
          Reporter: [e-mail]

Created attachment 66004
  --> https://bugs.winehq.org/attachment.cgi?id=66004
backtrace

Only happens when creating a 64bit prefix.

Attached is the backtrace, there was no other output aside from the regular
output when creating a prefix.

I'll try to do a regression test ASAP.

[Bug 48297] Wineboot crashes on 64bit prefix creation (macOS wine built without mingw)

[WineHQ Bugzilla]

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

Gijs Vermeulen  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]
    Regression SHA1|                            |9cf9483a4a0f33af6af0beb6739
                   |                            |5264f1c8ad259

--- Comment #1 from Gijs Vermeulen  ---
Bisection revealed:

9cf9483a4a0f33af6af0beb67395264f1c8ad259 is the first bad commit
commit 9cf9483a4a0f33af6af0beb67395264f1c8ad259
Author: Zebediah Figura 
Date:   Wed Nov 6 23:54:59 2019 -0600

wineboot: Create a root-enumerated device object for winebus.

Signed-off-by: Zebediah Figura 
    Signed-off-by: Alexandre Julliard

[Bug 48297] Wineboot crashes on 64bit prefix creation (macOS wine built without mingw)

[WineHQ Bugzilla]

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

--- Comment #2 from Alexandre Julliard  ---
It's probably because clang generates calls to memcpy, memset etc. using the
host calling convention. There doesn't seem to be a good way of avoiding that,
so it may require clang patches.

[Bug 48297] Wineboot crashes on 64bit prefix creation (macOS wine built without mingw)

[WineHQ Bugzilla]

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

Zebediah Figura  changed:

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

--- Comment #3 from Zebediah Figura  ---
(In reply to Alexandre Julliard from comment #2)
> It's probably because clang generates calls to memcpy, memset etc. using the
> host calling convention. There doesn't seem to be a good way of avoiding
> that, so it may require clang patches.

Looks like that's indeed the case:

000a:000b:Call
setupapi.SetupDiGetDeviceInstallParamsW(004f0f40,0021e7a0,0021e558)
ret=738e5a72
000a:000b:trace:setupapi:SetupDiGetDeviceInstallParamsW devinfo
00000000004F0F40, device_data 000000000021E7A0, params 000000000021E558.
000a:000b:Call msvcrt.memcpy(7fff693d4f4a,00000248,00000002) ret=7020ef04
000a:000b:Call kernelbase.UnhandledExceptionFilter(0021d4a0) ret=7b45f553

0x248 is the size of SP_DEVINSTALL_PARAMS; it's being put in %rdx when it
should actually be in %r8. Resolving upstream.

[Bug 48297] Wineboot crashes on 64bit prefix creation (macOS wine built without mingw)

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

--- Comment #4 from Alexandre Julliard  ---
*** Bug 48759 has been marked as a duplicate of this bug. ***