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.

Sign up for GitHub

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

Jump to bottom

[Android] ImportExport.ImportZeroModulus crypto test fails #79584

Closed
steveisok opened this issue Dec 13, 2022 · 2 comments · Fixed by #79585
Closed

[Android] ImportExport.ImportZeroModulus crypto test fails #79584

steveisok opened this issue Dec 13, 2022 · 2 comments · Fixed by #79585
Assignees
@vcsjones
Labels
area-System.Security os-android

Comments

@steveisok
Copy link
Member

#79013 added the ImportZeroModulus test and it's failing on Android arm64 with:

Assert.Throws() Failure
Expected: typeof(System.Security.Cryptography.CryptographicException)
Actual:   (No exception was thrown)

AzDo build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=109880&view=ms.vss-test-web.build-test-results-tab&runId=2343576&resultId=167385&paneView=debug
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Dec 13, 2022
@ghost
Copy link

ghost commented Dec 13, 2022

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

Issue Details

#79013 added the ImportZeroModulus test and it's failing on Android arm64 with:

Assert.Throws() Failure
Expected: typeof(System.Security.Cryptography.CryptographicException)
Actual:   (No exception was thrown)

AzDo build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=109880&view=ms.vss-test-web.build-test-results-tab&runId=2343576&resultId=167385&paneView=debug

Author: steveisok
Assignees: -
Labels:

area-System.Security
Milestone: -

@steveisok
Copy link
Member Author

/cc @vcsjones @simonrozsival

@vcsjones vcsjones mentioned this issue Dec 13, 2022
Merged
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Dec 13, 2022
@vcsjones vcsjones self-assigned this Dec 13, 2022
@ghost ghost removed in-pr There is an active PR which will close this issue when it is merged untriaged New issue has not been triaged by the area owner labels Dec 13, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Jan 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@vcsjones vcsjones

Labels
area-System.Security os-android
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Handle RSA zero modulus for Android vcsjones/runtime
2 participants
@vcsjones @steveisok