Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SCOSSL: Shake Squeeze tests fail #112036

Open
vcsjones opened this issue Jan 31, 2025 · 2 comments
Open

SCOSSL: Shake Squeeze tests fail #112036

vcsjones opened this issue Jan 31, 2025 · 2 comments
Labels
area-System.Security disabled-test The test is disabled in source code against the issue
Milestone

Comments

@vcsjones
Copy link
Member

For SymCrypt-OpenSSL (SCOSSL) the implementation of EVP_DigestSqueeze does not appear to work correctly which results in test failures like:

Interop+Crypto+OpenSslCryptographicException : error:1C80006A:Provider routines::output buffer too small
      Stack Trace:
        /_/src/libraries/System.Security.Cryptography/src/System/Security/Cryptography/LiteHash.Unix.cs(110,0): at System.Security.Cryptography.LiteXof.Check(Int32 result)
        /_/src/libraries/System.Security.Cryptography/src/System/Security/Cryptography/LiteHash.Unix.cs(95,0): at System.Security.Cryptography.LiteXof.Read(Span`1 destination)
        /_/src/libraries/System.Security.Cryptography/src/System/Security/Cryptography/Shake256.cs(216,0): at System.Security.Cryptography.Shake256.Read(Span`1 destination)
        /_/src/libraries/System.Security.Cryptography/tests/Shake256Tests.cs(24,0): at System.Security.Cryptography.Tests.Shake256Tests.Traits.Read(Shake256 shake, Span`1 destination)
           at System.Security.Cryptography.Tests.ShakeTestDriver`2.Read_MixedCloneAfterRead()

This issue is to track the disablement of the Shake Read/Squeeze tests.

@vcsjones vcsjones added area-System.Security disabled-test The test is disabled in source code against the issue labels Jan 31, 2025
@vcsjones vcsjones added this to the Future milestone Jan 31, 2025
Copy link
Contributor

Tagging subscribers to this area: @dotnet/area-system-security, @bartonjs, @vcsjones
See info in area-owners.md if you want to be subscribed.

@vcsjones
Copy link
Member Author

Changed milestone to 10. This is fixed in source, we just need to wait for new package builds to pick up the change, and our container images pick up the new packages. microsoft/SymCrypt-OpenSSL#105

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Security disabled-test The test is disabled in source code against the issue
Projects
None yet
Development

No branches or pull requests

1 participant