-
Notifications
You must be signed in to change notification settings - Fork 15k
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
fix: use WeakPtr to detect deletion to avoid crash on application menu #19257
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM on usage, but can you upstream this to get a better review ? we can pull back that patch, Thanks!
@robo upstream CL is here: https://chromium-review.googlesource.com/c/chromium/src/+/1714626 |
2a8506a
to
77c24d8
Compare
@robo the upstream patch has been approved. I updated this PR with the feedback provided in the CL. |
Release Notes Persisted
|
I was unable to backport this PR to "6-0-x" cleanly; |
I was unable to backport this PR to "5-0-x" cleanly; |
I have automatically backported this PR to "7-0-x", please check out #19558 |
A maintainer has manually backported this PR to "6-0-x", please check out #19560 |
A maintainer has manually backported this PR to "5-0-x", please check out #19561 |
Description of Change
Fixes #18436. This PR resolves a
heap-use-after-free
issue found by the AddressSanitizer. This issue happens when interacting with the Application Menu.Checklist
npm test
passesRelease Notes
Notes: Fixed crashes when using the application menu on Linux.