mirrors/gst-plugins-rs
mirrors/gst-plugins-rs
1
1
Fork 0
mirror of https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs.git synced 2024-06-20 23:20:36 +00:00
Code Issues Projects Releases Wiki Activity

webrtcsink: Refactor value retrieval to avoid lock poisoning

Browse source 
When setting an incorrect property name in settings,
start_stream_discovery_if_needed would panic because it attempts to
unwrap a poisoned lock for settings.

This refactor avoids that situation.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-rs/-/merge_requests/1594>
This commit is contained in:
Francisco Javier Velázquez-García 2024-05-29 18:28:02 +03:00 committed by GStreamer Marge Bot
parent 568e8533fa
commit 8fc652f208
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
net/webrtc/src/webrtcsink/imp.rs
View file

@ -4240,8 +4240,8 @@ impl ObjectImpl for BaseWebRTCSink {
);
let this = element.imp();
let settings = this.settings.lock().unwrap();
configure_encoder(&enc, settings.cc_info.start_bitrate);
let start_bitrate = this.settings.lock().unwrap().cc_info.start_bitrate;
configure_encoder(&enc, start_bitrate);
// Return false here so that latter handlers get called
Some(false.to_value())