From f292e3151155d90ca6dd9d0a1a51b4de67c02572 Mon Sep 17 00:00:00 2001 From: danilchap Date: Thu, 22 Sep 2016 07:24:30 -0700 Subject: [PATCH] Relax too strict DCHECKs while parsing rtcp reports BUG=chromium:649129 Review-Url: https://codereview.webrtc.org/2361493004 Cr-Commit-Position: refs/heads/master@{#14353} --- webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report.cc | 2 +- webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report.cc b/webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report.cc index 10744ad939..a393181cea 100644 --- a/webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report.cc +++ b/webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report.cc @@ -50,7 +50,7 @@ bool ReceiverReport::Parse(const CommonHeader& packet) { next_report_block += ReportBlock::kLength; } - RTC_DCHECK_EQ(next_report_block - packet.payload(), + RTC_DCHECK_LE(next_report_block - packet.payload(), static_cast(packet.payload_size_bytes())); return true; } diff --git a/webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc b/webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc index 8f90ce6bdd..4795bf16e5 100644 --- a/webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc +++ b/webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc @@ -68,7 +68,7 @@ bool SenderReport::Parse(const CommonHeader& packet) { next_block += ReportBlock::kLength; } // Double check we didn't read beyond provided buffer. - RTC_DCHECK_EQ(next_block - payload, + RTC_DCHECK_LE(next_block - payload, static_cast(packet.payload_size_bytes())); return true; }