mirrors/gst-plugins-rs
mirrors/gst-plugins-rs
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs.git synced 2025-02-15 12:25:20 +00:00
Code Issues Projects Releases Wiki Activity

rtpgccbwe: Don't process empty lists

Browse source 
The structure parsing could result in an empty vector. Don't do any processing
since the loss code assumes it's non-empty for average estimates which would
result in weird/invalid results.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs/-/merge_requests/1190>
This commit is contained in:
Edward Hervey 2023-04-15 08:17:09 +02:00 committed by Sebastian Dröge
parent 5462ed0e95
commit f4a565d4ea
1 changed files with 14 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
net/rtp/src/gcc/imp.rs
View file

@ -1194,6 +1194,8 @@ impl ObjectSubclass for BandwidthEstimator {
}) })
.collect::<Vec<Packet>>(); .collect::<Vec<Packet>>();
// The list of packets could be empty once parsed
if !packets.is_empty() {
let bitrate_changed = { let bitrate_changed = {
let mut state = this.state.lock().unwrap(); let mut state = this.state.lock().unwrap();
@ -1210,6 +1212,7 @@ impl ObjectSubclass for BandwidthEstimator {
} }
} }
} }
}
gst::Pad::event_default(pad, parent, event) gst::Pad::event_default(pad, parent, event)
}, },