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 e0efbd45ea74 from webrtc #37299

Merged
merged 2 commits into from
Feb 20, 2023

Conversation

VerteDinde
Copy link
Member

@VerteDinde VerteDinde commented Feb 16, 2023

[Merge-110] [Stats] Handle the case of missing certificates.

Certificates being missing is a sign of a bug (e.g. webrtc:14844, to be
fixed separately) which is why we have a DCHECK. But this DCHECK does
not protect against accessing the invalid iterator if it is a release
build. This CL makes that safe.

(cherry picked from commit 124d7c3fe5bdc79a355c9df02d07f25331631a68)

Bug: chromium:1408392
Change-Id: I97a82786028e41c58ef8ef15002c3f959bbec7f1
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/291109
Reviewed-by: Mirko Bonadei mbonadei@webrtc.org
Commit-Queue: Henrik Boström hbos@webrtc.org
Cr-Original-Commit-Position: refs/heads/main@{#39159}
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/291380
Cr-Commit-Position: refs/branch-heads/5481@{#2}
Cr-Branched-From: 2e1a9a4ae0234d4b1ea7a6fd4188afa1fb20379d-refs/heads/main@{#38901}

Notes: Security: backported fix for chromium:1408392.

@VerteDinde VerteDinde requested review from a team as code owners February 16, 2023 06:19
@VerteDinde VerteDinde added security 🔒 semver/patch backwards-compatible bug fixes backport-check-skip Skip trop's backport validity checking 21-x-y labels Feb 16, 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 Feb 16, 2023
@codebytere codebytere merged commit 334627c into 21-x-y Feb 20, 2023
@codebytere codebytere deleted the cherry-pick/21-x-y/webrtc/e0efbd45ea74 branch February 20, 2023 12:13
@release-clerk
Copy link

release-clerk bot commented Feb 20, 2023

Release Notes Persisted

Security: backported fix for chromium:1408392.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
21-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

3 participants