[Bug 57117] New: BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

Bug ID: 57117
           Summary: BsgLauncher cannot launch due to unhandled exception
                    in System.Security.Cryptography (.NET48)
           Product: Wine
           Version: 9.16
          Hardware: x86-64
                OS: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
          Assignee: [e-mail]
          Reporter: [e-mail]
      Distribution: ---

Created attachment 77009
  --> https://bugs.winehq.org/attachment.cgi?id=77009
error log, used WINEDEBUG=+seh+debugstr

Upon launch, the BsgLauncher used to play Escape from Tarkov / Escape from
Tarkov: Arena, crashes early on (no window actually appears), stating an error
that might be related to the following err line:

System.Security.Cryptography.Utils._DecryptData(System.Security.Cryptography.SafeKeyHandle,
Byte[], Int32, Int32, Byte[] ByRef, Int32,
System.Security.Cryptography.PaddingMode, Boolean)\n"

See attached log file for more details.

- This issue has been introduced with Wine 9.15.
- Earlier versions using the same dependencies won't fail with that error
(working state)
- Switching between different wine version (win81, win10, win11) had no effect
for me

BsgLauncher: https://prod.escapefromtarkov.com/launcher/download
Prefix dependencies: arial times vcrun2019 dotnet48

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

--- Comment #1 from Lars Loe  ---
Only commit in Wine 9.15 release I could find that has something do to with
cryptgen is this one:

https://gitlab.winehq.org/wine/wine/-/commit/9f0edb69209eb00faac55bbf1a5f605...

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

Lars Loe  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                URL|                            |https://prod.escapefromtark
                   |                            |ov.com/launcher/download
       Distribution|---                         |Fedora

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

Lars Loe  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

--- Comment #2 from Ken Sharp  ---
Does reverting that commit help?
https://wiki.winehq.org/Regression_Testing

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
    Regression SHA1|                            |9f0edb69209eb00faac55bbf1a5
                   |                            |f605750517201
             Status|UNCONFIRMED                 |NEW
           Keywords|                            |regression

--- Comment #3 from Dmitry Timoshkov  ---
(In reply to Lars Loe from comment #1)
> Only commit in Wine 9.15 release I could find that has something do to with
> cryptgen is this one:
>
> https://gitlab.winehq.org/wine/wine/-/commit/
> 9f0edb69209eb00faac55bbf1a5f605750517201

Can confirm that either reverting this commit or replacing
if (!*pdwDataLen)
{
    SetLastError(NTE_BAD_LEN);
    return FALSE;
}
by
if (!*pdwDataLen) return TRUE;
fixes the crash.

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

--- Comment #4 from Hans Leidekker  ---
(In reply to Dmitry Timoshkov from comment #3)
> (In reply to Lars Loe from comment #1)
> > Only commit in Wine 9.15 release I could find that has something do to with
> > cryptgen is this one:
> >
> > https://gitlab.winehq.org/wine/wine/-/commit/
> > 9f0edb69209eb00faac55bbf1a5f605750517201
>
> Can confirm that either reverting this commit or replacing
> if (!*pdwDataLen)
> {
>     SetLastError(NTE_BAD_LEN);
>     return FALSE;
> }
> by
> if (!*pdwDataLen) return TRUE;
> fixes the crash.

I have submitted an MR:
https://gitlab.winehq.org/wine/wine/-/merge_requests/6368

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

Lars Loe  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|-unknown                    |rsaenh

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

Hans Leidekker  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Fixed by SHA1|                            |c2591522edaa4cfea14821c74c5
                   |                            |d8f4535d5296e
             Status|NEW                         |RESOLVED

--- Comment #5 from Hans Leidekker  ---
Should be fixed by c2591522edaa4cfea14821c74c5d8f4535d5296e.

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

--- Comment #6 from Lars Loe  ---
Thanks for testing Dimitry & thanks Hans for the quick fix.

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

--- Comment #7 from Hans Leidekker  ---
(In reply to Lars Loe from comment #6)
> Thanks for testing Dimitry & thanks Hans for the quick fix.

Thank you for the bug report.

[Bug 57117] BsgLauncher cannot launch due to unhandled exception in System.Security.Cryptography (.NET48)

[WineHQ Bugzilla]

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

Alexandre Julliard  changed:

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

--- Comment #8 from Alexandre Julliard  ---
Closing bugs fixed in 9.17.