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

X509CertificateLoader fails to read cat files #109046

Open
Nuklon opened this issue Oct 19, 2024 · 3 comments
Open

X509CertificateLoader fails to read cat files #109046

Nuklon opened this issue Oct 19, 2024 · 3 comments
Labels
area-System.Security needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration
Milestone

Comments

@Nuklon
Copy link

Nuklon commented Oct 19, 2024

Description

The now obsoleted ctor works fine when reading cat files, but the new X509CertificateLoader fails to load it and throws "Cannot find the requested object.".

Reproduction Steps

using X509Certificate2 certificate = X509CertificateLoader.LoadCertificateFromFile(path); // throws

Example cat file (taken from latest AMD driver, rename to .cat): example.cat.txt

Expected behavior

Load OK.

Actual behavior

Throws an exception.

Regression?

Yes.

Known Workarounds

Use obsoleted ctor.

Configuration

.NET 9.0 RC2
Windows 11 24H2 x64

Other information

No response

@dotnet-policy-service dotnet-policy-service bot added the untriaged New issue has not been triaged by the area owner label Oct 19, 2024
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.

@jeffhandley
Copy link
Member

@bartonjs, as far as I know we didn't consider CAT files to be in scope for the new X509CertificateLoader; is that right? From my understanding, they aren't actually X.509 certificate payloads.

@Nuklon, what data were you extracting from the CAT file via the X509Certificate2 API?

@jeffhandley jeffhandley added this to the Future milestone Oct 20, 2024
@jeffhandley jeffhandley added needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration and removed untriaged New issue has not been triaged by the area owner labels Oct 20, 2024
@Nuklon
Copy link
Author

Nuklon commented Oct 20, 2024

@Nuklon, what data were you extracting from the CAT file via the X509Certificate2 API?

The issuer and expiration date.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Security needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration
Projects
None yet
Development

No branches or pull requests

2 participants