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-16 15:06:43 +0000
commitbe349eaf62e77955791d7bf29f893f1e8a37daf4 (patch)
tree576f1815a8007894f7271a8187fd0b6664314daa
parent9d42f3a5802d427c0e9358b2a0469f18285558c0 (diff)
downloadqtwebengine-chromium-be349eaf62e77955791d7bf29f893f1e8a37daf4.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 be25e995c67..a7594e80ad1 100644
--- a/chromium/third_party/webrtc/pc/rtp_sender.cc
+++ b/chromium/third_party/webrtc/pc/rtp_sender.cc
@@ -70,8 +70,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;