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

Check for controller in SSCC when route changed.

Browse Source 
Fixes potential crash in SendSideCongestionController when route is
changed before network is available.

Bug: webrtc:8415
Change-Id: I781f0e342e5bb42fedbf96c9c5c6d2c199ab3192
Reviewed-on: https://webrtc-review.googlesource.com/69801
Reviewed-by: Björn Terelius <terelius@webrtc.org>
Commit-Queue: Sebastian Jansson <srte@webrtc.org>
Cr-Commit-Position: refs/heads/master@{#22851}
This commit is contained in:
Sebastian Jansson
2018-04-13 10:46:26 +02:00
committed by Commit Bot
parent 548b771a80
commit 70488de694
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
modules/congestion_controller/rtp/send_side_congestion_controller.cc
View File

@ -480,7 +480,12 @@ void SendSideCongestionController::OnNetworkRouteChanged(
start_bitrate_bps > 0 ? DataRate::bps(start_bitrate_bps) : DataRate();
task_queue_->PostTask([this, msg]() {
RTC_DCHECK_RUN_ON(task_queue_ptr_);
PostUpdates(controller_->OnNetworkRouteChange(msg));
if (controller_) {
PostUpdates(controller_->OnNetworkRouteChange(msg));
} else {
initial_config_.starting_bandwidth = msg.starting_rate;
initial_config_.constraints = msg.constraints;
}
pacer_controller_->OnNetworkRouteChange(msg);
});
}