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

chore: cherry-pick 1 changes from Release-2-M119 #40537

Merged
merged 2 commits into from Nov 20, 2023

Conversation

ppontes
Copy link
Member

@ppontes ppontes commented Nov 15, 2023

electron/security#427 - 9384cddc7705 from chromium [Merge to M118] Make URLSearchParams persistent to avoid UaF

The URLSearchParams::Create() function returns an on-heap object, but it
can be garbage collected, so making it a persistent variable in
DidFetchDataLoadedString() mitigates the issue.

(cherry picked from commit 8b1bd7726a1394e2fe287f6a882822d8ee9d4e96)

Bug: 1497997
Change-Id: I4ae0f93fccc561cd8a088d3fa0bf2968bf298acf
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/4996929
Reviewed-by: Adam Rice ricea@chromium.org
Commit-Queue: Nidhi Jaju nidhijaju@chromium.org
Cr-Original-Commit-Position: refs/heads/main@{#1218682}
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5007484
Commit-Queue: Adam Rice ricea@chromium.org
Auto-Submit: Nidhi Jaju nidhijaju@chromium.org
Cr-Commit-Position: refs/branch-heads/5993@{#1546}
Cr-Branched-From: 511350718e646be62331ae9d7213d10ec320d514-refs/heads/main@{#1192594}

Notes:

@ppontes ppontes requested a review from a team as a code owner November 15, 2023 22:56
@ppontes ppontes added security 🔒 semver/patch backwards-compatible bug fixes backport-check-skip Skip trop's backport validity checking 26-x-y labels Nov 15, 2023
@electron-cation electron-cation bot added new-pr 🌱 PR opened in the last 24 hours and removed new-pr 🌱 PR opened in the last 24 hours labels Nov 15, 2023
@zcbenz zcbenz merged commit ead3de1 into 26-x-y Nov 20, 2023
12 of 13 checks passed
@zcbenz zcbenz deleted the cherry-pick/security/26-x-y/release-2-m119 branch November 20, 2023 00:00
Copy link

release-clerk bot commented Nov 20, 2023

Release Notes Persisted

  • Security: backported fix for CVE-2023-5997.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
26-x-y backport-check-skip Skip trop's backport validity checking security 🔒 semver/patch backwards-compatible bug fixes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants