admin/webrtc_m130
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Relax too strict DCHECKs while parsing rtcp reports

Browse Source 
BUG=chromium:649129

Review-Url: https://codereview.webrtc.org/2361493004
Cr-Commit-Position: refs/heads/master@{#14353}
This commit is contained in:
danilchap 2016-09-22 07:24:30 -07:00 committed by Commit bot
parent aac9d6fb25
commit f292e31511
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
webrtc/modules/rtp_rtcp/source/rtcp_packet/receiver_report.cc
View File

@ -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<ptrdiff_t>(packet.payload_size_bytes()));
return true;
}

2
webrtc/modules/rtp_rtcp/source/rtcp_packet/sender_report.cc
View File

@ -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<ptrdiff_t>(packet.payload_size_bytes()));
return true;
}