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-11-29 15:01:07 +00:00
Code Issues Projects Releases Wiki Activity

ffv1dec: Drop decoder state before calling VideoDecoder::negotiate()

Browse source 
It calls into other elements via the allocation query and the caps
event, which makes it dangerous to keep a mutex locked at the same time.
This commit is contained in:
Sebastian Dröge 2021-09-18 10:55:26 +03:00
parent 4c8c398aa4
commit fd1c0a7518
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
video/ffv1/src/ffv1dec/imp.rs
View file

@ -304,15 +304,17 @@ impl VideoDecoderImpl for Ffv1Dec {
let output_info = Some(output_state.info());
element
.negotiate(output_state)
.map_err(|err| gst::loggable_error!(CAT, "Negotiation failed: {}", err))?;
let mut decoder_state = self.state.lock().unwrap();
*decoder_state = DecoderState::Started {
output_info,
decoder: Box::new(decoder),
video_meta_supported: false,
};
drop(decoder_state);
element
.negotiate(output_state)
.map_err(|err| gst::loggable_error!(CAT, "Negotiation failed: {}", err))?;
self.parent_set_format(element, state)
}