Skip to content

Conversation

geoffw0
Copy link
Contributor

@geoffw0 geoffw0 commented Jul 23, 2024

More tests for cpp/use-after-free. This was a second failed attempt to reproduce an issue I've seen once in the wild. I'm going to cut my losses here.

@geoffw0 geoffw0 added the C++ label Jul 23, 2024
@geoffw0 geoffw0 requested a review from a team as a code owner July 23, 2024 17:06
@geoffw0 geoffw0 added the no-change-note-required This PR does not need a change note label Jul 23, 2024
Copy link
Contributor

@MathiasVP MathiasVP left a comment

Choose a reason for hiding this comment

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

LGTM! Note that the FP may have been fixed by #16749

@geoffw0 geoffw0 merged commit 741a328 into github:main Jul 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ no-change-note-required This PR does not need a change note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants