zqz/platform-external-webrtc
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Adds check for uninitialized window start sequence.

Browse Source 
Bug: webrtc:9812
Change-Id: I34d1797491b83ea7a106418fbd24e04893891559
Reviewed-on: https://webrtc-review.googlesource.com/c/103660
Reviewed-by: Stefan Holmer <stefan@webrtc.org>
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#24978}
This commit is contained in:
Sebastian Jansson
2018-10-04 10:08:03 +02:00
committed by Commit Bot
parent bc2959072d
commit 5186582616
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
modules/remote_bitrate_estimator/remote_estimator_proxy.cc
View File

@ -122,7 +122,7 @@ void RemoteEstimatorProxy::OnPacketArrival(uint16_t sequence_number,
// SequenceNumberUnwrapper doesn't do this, so we should replace this with
// calls to IsNewerSequenceNumber instead.
int64_t seq = unwrapper_.Unwrap(sequence_number);
if (seq > window_start_seq_ + 0xFFFF / 2) {
if (window_start_seq_ != -1 && seq > window_start_seq_ + 0xFFFF / 2) {
RTC_LOG(LS_WARNING) << "Skipping this sequence number (" << sequence_number
<< ") since it likely is reordered, but the unwrapper"
"failed to handle it. Feedback window starts at "