From 51865826165faeb5f5096e232245b22b7cc4e27d Mon Sep 17 00:00:00 2001 From: Sebastian Jansson Date: Thu, 4 Oct 2018 10:08:03 +0200 Subject: [PATCH] Adds check for uninitialized window start sequence. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Bug: webrtc:9812 Change-Id: I34d1797491b83ea7a106418fbd24e04893891559 Reviewed-on: https://webrtc-review.googlesource.com/c/103660 Reviewed-by: Stefan Holmer Reviewed-by: Björn Terelius Commit-Queue: Sebastian Jansson Cr-Commit-Position: refs/heads/master@{#24978} --- modules/remote_bitrate_estimator/remote_estimator_proxy.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/modules/remote_bitrate_estimator/remote_estimator_proxy.cc b/modules/remote_bitrate_estimator/remote_estimator_proxy.cc index 8cde28d027..d9745c9f54 100644 --- a/modules/remote_bitrate_estimator/remote_estimator_proxy.cc +++ b/modules/remote_bitrate_estimator/remote_estimator_proxy.cc @@ -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 "