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

Ensure a null PublicKey is supported on assembly refs #108907

Merged
merged 1 commit into from
Oct 16, 2024

Conversation

steveharter
Copy link
Member

Fixes #108517.

Regression from #95536 which was done early in v9. If the referenced assembly had no private key, the uninitialized StrongNameToken variable was used as the PublicKey which would contain random values instead of the expected null.

This should be ported to v9.

jkoritzinsky added a commit to jkoritzinsky/DNMD that referenced this pull request Oct 15, 2024
@steveharter steveharter merged commit 7056bbd into dotnet:main Oct 16, 2024
88 checks passed
@steveharter steveharter deleted the Issue_108517 branch October 16, 2024 13:19
@steveharter
Copy link
Member Author

/backport to release/9.0

Copy link
Contributor

Started backporting to release/9.0: https://github.com/dotnet/runtime/actions/runs/11366596971

jkoritzinsky added a commit to AaronRobinsonMSFT/DNMD that referenced this pull request Oct 16, 2024
Co-authored-by: Aaron Robinson <arobins@microsoft.com>
jkoritzinsky added a commit to jkoritzinsky/runtime that referenced this pull request Oct 31, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Nov 16, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Regression in net9.0 when calling a ServiceClient from a dynamically loaded Assembly
3 participants