Skip to content

Conversation

@nornagon
Copy link
Contributor

@nornagon nornagon commented Apr 8, 2020

Use SupportsWeakPtr for messaging from rendering thread to main thread

In cross-thread messaging, the associated execution context can be
already gone when a posted task is performed sometime later in the task
runner's queue.

By using WeakPtr, the task runner will not perform a scheduled task
in the queue when the target object is invalid.

Test: Locally confirmed that the repro does not crash.
Bug: 1057627
Change-Id: I51737594c918f6a4924c9a7ffe30db3e8de9a683
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2082897
Commit-Queue: Hongchan Choi hongchan@chromium.org
Reviewed-by: Raymond Toy rtoy@chromium.org
Cr-Commit-Position: refs/heads/master@{#746613}

Notes: Security: backported fix for CVE-2020-6429: Use after free in audio.

@nornagon nornagon requested a review from a team as a code owner April 8, 2020 00:40
@electron-cation electron-cation bot added the new-pr 🌱 PR opened recently label Apr 8, 2020
@nornagon nornagon added 7-2-x backport-check-skip Skip trop's backport validity checking labels Apr 8, 2020
@electron-cation electron-cation bot added new-pr 🌱 PR opened recently and removed new-pr 🌱 PR opened recently labels Apr 8, 2020
@nornagon nornagon merged commit 3bf5ad2 into 7-2-x Apr 10, 2020
@release-clerk
Copy link

release-clerk bot commented Apr 10, 2020

Release Notes Persisted

Security: backported fix for CVE-2020-6429: Use after free in audio.

@nornagon nornagon deleted the cherry-pick/7-2-x/chromium/913247c378d5 branch April 10, 2020 00:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

7-2-x backport-check-skip Skip trop's backport validity checking

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants