[Bug 53968] New: NCryptSignHash is not implemented
https://bugs.winehq.org/show_bug.cgi?id=53968 Bug ID: 53968 Summary: NCryptSignHash is not implemented Product: Wine Version: 7.21 Hardware: x86-64 OS: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: bcrypt Assignee: wine-bugs(a)winehq.org Reporter: jesper.lindholm(a)treetop.se Distribution: --- Created attachment 73557 --> https://bugs.winehq.org/attachment.cgi?id=73557 Terminal output Using Ubuntu 22.04 x64, from-source build as of today (wine-7.21) + fix from bug 53966. Please note that the patch from 53996 is required to get this far in the reproduction. The .NET API RSA.SignData() calls into NCryptSignHash which is not implemented. Using .NET 6.0. Attaching terminal output. A minimal reproducible example is at https://treetop.se/wine-bug/2022-11-23b/source.zip (a handful of lines of code) , compiled as https://treetop.se/wine-bug/2022-11-23b/wine-missing-apis.zip (published as "self-contained", ie bundling the framework which I thought might be helpful here) . compiled .zip file hashes: SHA256 1F881B2C1946C276B6F602F8EEC11FF88B82C99EAAA39A6F13EB9173EA403D95 SHA1 B02F53D648DB20A680B5E93491D8094278FE2285 source .zip file hashes: SHA256 3977B2AA462DC54AEEBFB964FBED17A5AAA8360C501CEA3060AB239C09BBD307 SHA1 9B3E768B4F156E17B8E317C82599F9E7FE225C56 This API is used in many applications signing messages with their own cryptographic RSA keys. We ran across it when testing one of our own applications. ncrypt looks like it should be its own component due to being its own DLL, but I can't find it or anything that looks like it; filing under bcrypt due to ncrypt implementation calling through to bcrypt in places in the hopes that it will reach the right people. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=53968 --- Comment #1 from Hans Leidekker <hans(a)meelstraat.net> --- Created attachment 73558 --> https://bugs.winehq.org/attachment.cgi?id=73558 patch Here's a patch that implements it on top of BCryptSignHash(). It's not sufficient for your test app because it passes the BCRYPT_PAD_PSS flag, which is unimplemented. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=53968 --- Comment #2 from jesper.lindholm(a)treetop.se --- That's great. Should we be filing a separate bug for that or is that work ongoing/implicitly logged? Your comment was the only mention I found for both BCRYPT_PAD_PSS and BCryptSignHash. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=53968 --- Comment #3 from Hans Leidekker <hans(a)meelstraat.net> --- (In reply to jesper.lindholm from comment #2)
That's great. Should we be filing a separate bug for that or is that work ongoing/implicitly logged? Your comment was the only mention I found for both BCRYPT_PAD_PSS and BCryptSignHash.
Yes, please file a separate bug and I'll take a look. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=53968 --- Comment #4 from jesper.lindholm(a)treetop.se --- Filed as bug 53972 - it refers back to this for details on reproducing it. I hope that's okay. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=53968 Hans Leidekker <hans(a)meelstraat.net> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |FIXED Fixed by SHA1| |6d01d014552782c55d297f5244b | |e01a11c33d35e Status|UNCONFIRMED |RESOLVED --- Comment #5 from Hans Leidekker <hans(a)meelstraat.net> --- Fixed with 6d01d014552782c55d297f5244be01a11c33d35e. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=53968 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #6 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 8.0-rc1. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
WineHQ Bugzilla