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 adc8f05aa3ab from chromium #22977

Merged
merged 2 commits into from
Apr 8, 2020

Conversation

nornagon
Copy link
Member

@nornagon nornagon commented Apr 6, 2020

Handle ERR_CACHE_RACE in DoDoneHeadersAddToEntryComplete

Crashes have been observed in the wild that may be due to
ERR_CACHE_RACE being sent to STATE_DONE_HEADERS_ADD_TO_ENTRY_COMPLETE.
These crashes, however, cannot be reproduced and code investigation
does not suggest how they might happen. If the crashes stop occurring
following this change, we will be sure that this is the cause and will
perform more analysis based on that understanding.

Bug: 959194
Change-Id: I01649ee31024b598e41c0beda7c7ed54980cb133
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1814843
Commit-Queue: Alex Turner alexmt@chromium.org
Reviewed-by: Shivani Sharma shivanisha@chromium.org
Cr-Commit-Position: refs/heads/master@{#698597}

Notes: none

@nornagon nornagon requested a review from a team as a code owner April 6, 2020 19:22
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Apr 6, 2020
@nornagon nornagon added the 7-2-x label Apr 6, 2020
@electron-cation electron-cation bot added the new-pr 🌱 PR opened in the last 24 hours label Apr 6, 2020
@nornagon nornagon added the backport-check-skip Skip trop's backport validity checking label Apr 6, 2020
@nornagon nornagon requested a review from a team April 7, 2020 18:08
@electron-cation electron-cation bot removed the new-pr 🌱 PR opened in the last 24 hours label Apr 7, 2020
@jkleinsc
Copy link
Contributor

jkleinsc commented Apr 8, 2020

Merging as CI failures are flakes unrelated to PR change.

@jkleinsc jkleinsc merged commit 5ed3bd8 into 7-2-x Apr 8, 2020
@release-clerk
Copy link

release-clerk bot commented Apr 8, 2020

Release Notes Persisted

Security: Backport fix for use after free in http cache.

@jkleinsc jkleinsc deleted the cherry-pick/7-2-x/chromium/adc8f05aa3ab branch April 8, 2020 14:53
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.

None yet

3 participants