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

Don't report packets with id -1 to the transport feedback adapter as they provide no value.

Browse Source 
This also avoids an issue where -1 is interpreted as 0xFFFF by the feedback adapter, which in practice likely isn't a problem, but still wrong.

BUG=None

Review-Url: https://codereview.webrtc.org/2579263002
Cr-Commit-Position: refs/heads/master@{#15647}
This commit is contained in:
stefan 2016-12-16 02:11:26 -08:00 committed by Commit bot
parent 05d6e260f5
commit 528ec3db0b
1 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
webrtc/modules/congestion_controller/congestion_controller.cc
View File

@ -290,6 +290,10 @@ void CongestionController::SignalNetworkState(NetworkState state) {
}
void CongestionController::OnSentPacket(const rtc::SentPacket& sent_packet) {
// We're not interested in packets without an id, which may be stun packets,
// etc, sent on the same transport.
if (sent_packet.packet_id == -1)
return;
transport_feedback_adapter_.OnSentPacket(sent_packet.packet_id,
sent_packet.send_time_ms);
}