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

[release/9.0] Remove invalid assert in RSAOpenSsl #107952

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Sep 17, 2024

Backport of #107932 to release/9.0

/cc @vcsjones

Customer Impact

  • Customer reported
  • Found internally

Regression

  • Yes
  • No

Failing test reported in #107929 and #105715. An assert existed that a particular SafeHandle is valid, however this is not true in a multi-threaded case where the handle is disposed and used concurrently. This assert would occasionally be hit in unit tests.

SafeHandles handle thread safety, correctly, on their own. We should remove the assert and let the ObjectDisposedException from the SafeHandle happen.

Testing

Existing tests continue to pass.

Risk

None / Low. The change removes a Debug.Assert which is only present in Debug builds of Libraries.

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 vcsjones added this to the 9.0.0 milestone Sep 17, 2024
@jeffhandley jeffhandley added the Servicing-consider Issue for next servicing release review label Sep 17, 2024
@jeffhandley jeffhandley added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Sep 18, 2024
Copy link
Member

@carlossanlop carlossanlop left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was a fast fix!

@carlossanlop carlossanlop merged commit f1b67af into release/9.0 Sep 19, 2024
86 of 94 checks passed
@carlossanlop carlossanlop deleted the backport/pr-107932-to-release/9.0 branch September 19, 2024 01:52
@github-actions github-actions bot locked and limited conversation to collaborators Oct 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Security Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants