summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorHenrik Boström <hbos@webrtc.org>2022-07-13 11:10:14 +0200
committerMichael Brüning <michael.bruning@qt.io>2022-08-09 11:36:14 +0000
commit4b6ec8b20c9bcf0ecc0ee9ee174e50282f696011 (patch)
tree603c03cf6af9d7ab06eb1a98fbde9d74ba829f20
parentd750645e434057ae3d8426eed931fd48d32c1ed7 (diff)
downloadqtwebengine-chromium-4b6ec8b20c9bcf0ecc0ee9ee174e50282f696011.tar.gz
[Backport] Security bug 1343889
Cherry-pick of patch originally reviewed on https://webrtc-review.googlesource.com/c/src/+/268460: Disallow invalid arguments in RestoreEncodingLayers. Changing DCHECK into CHECK for good measure. Bug: chromium:1343889 Change-Id: I2cede85dc2d2a4238739f73afe25275047f4aa50 Reviewed-by: Ilya Nikolaevskiy <ilnik@webrtc.org> Commit-Queue: Henrik Boström <hbos@webrtc.org> Cr-Commit-Position: refs/heads/main@{#37511} Reviewed-by: Allan Sandfeld Jensen <allan.jensen@qt.io>
-rw-r--r--chromium/third_party/webrtc/pc/rtp_sender.cc4
1 files changed, 2 insertions, 2 deletions
diff --git a/chromium/third_party/webrtc/pc/rtp_sender.cc b/chromium/third_party/webrtc/pc/rtp_sender.cc
index 6b4fa3c039d..d52a53b9017 100644
--- a/chromium/third_party/webrtc/pc/rtp_sender.cc
+++ b/chromium/third_party/webrtc/pc/rtp_sender.cc
@@ -74,8 +74,8 @@ RtpParameters RestoreEncodingLayers(
const RtpParameters& parameters,
const std::vector<std::string>& removed_rids,
const std::vector<RtpEncodingParameters>& all_layers) {
- RTC_DCHECK_EQ(parameters.encodings.size() + removed_rids.size(),
- all_layers.size());
+ RTC_CHECK_EQ(parameters.encodings.size() + removed_rids.size(),
+ all_layers.size());
RtpParameters result(parameters);
result.encodings.clear();
size_t index = 0;