mirror of
https://gitlab.freedesktop.org/gstreamer/gstreamer.git
synced 2024-12-23 00:36:51 +00:00
decodebin3: avoid identity, sinkpad, parsebin leakage when reset input
when reset_input, need remove identity/parsebin from decodebin3 when release_pad, need call free or reset input if collection didn't change Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer/-/merge_requests/4664>
This commit is contained in:
parent
639367f818
commit
9f67b866b9
1 changed files with 8 additions and 26 deletions
|
@ -1129,37 +1129,17 @@ gst_decodebin3_release_pad (GstElement * element, GstPad * pad)
|
||||||
gst_element_post_message (GST_ELEMENT_CAST (dbin), msg);
|
gst_element_post_message (GST_ELEMENT_CAST (dbin), msg);
|
||||||
update_requested_selection (dbin);
|
update_requested_selection (dbin);
|
||||||
|
|
||||||
gst_ghost_pad_set_target (GST_GHOST_PAD (input->ghost_sink), NULL);
|
|
||||||
if (input->parsebin) {
|
if (input->parsebin) {
|
||||||
gst_bin_remove (GST_BIN (dbin), input->parsebin);
|
|
||||||
gst_element_set_state (input->parsebin, GST_STATE_NULL);
|
|
||||||
g_signal_handler_disconnect (input->parsebin, input->pad_removed_sigid);
|
|
||||||
g_signal_handler_disconnect (input->parsebin, input->pad_added_sigid);
|
|
||||||
g_signal_handler_disconnect (input->parsebin, input->drained_sigid);
|
|
||||||
gst_pad_remove_probe (input->parsebin_sink, probe_id);
|
gst_pad_remove_probe (input->parsebin_sink, probe_id);
|
||||||
gst_object_unref (input->parsebin);
|
|
||||||
gst_object_unref (input->parsebin_sink);
|
|
||||||
|
|
||||||
input->parsebin = NULL;
|
|
||||||
input->parsebin_sink = NULL;
|
|
||||||
}
|
|
||||||
if (input->identity) {
|
|
||||||
GstPad *idpad = gst_element_get_static_pad (input->identity, "src");
|
|
||||||
DecodebinInputStream *stream = find_input_stream_for_pad (dbin, idpad);
|
|
||||||
gst_object_unref (idpad);
|
|
||||||
remove_input_stream (dbin, stream);
|
|
||||||
gst_element_set_state (input->identity, GST_STATE_NULL);
|
|
||||||
gst_bin_remove (GST_BIN (dbin), input->identity);
|
|
||||||
gst_object_unref (input->identity);
|
|
||||||
input->identity = NULL;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!input->is_main) {
|
|
||||||
dbin->other_inputs = g_list_remove (dbin->other_inputs, input);
|
|
||||||
free_input (dbin, input);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
beach:
|
beach:
|
||||||
|
if (!input->is_main) {
|
||||||
|
dbin->other_inputs = g_list_remove (dbin->other_inputs, input);
|
||||||
|
free_input (dbin, input);
|
||||||
|
} else
|
||||||
|
reset_input (dbin, input);
|
||||||
|
|
||||||
INPUT_UNLOCK (dbin);
|
INPUT_UNLOCK (dbin);
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
@ -1177,6 +1157,7 @@ reset_input (GstDecodebin3 * dbin, DecodebinInput * input)
|
||||||
g_signal_handler_disconnect (input->parsebin, input->pad_added_sigid);
|
g_signal_handler_disconnect (input->parsebin, input->pad_added_sigid);
|
||||||
g_signal_handler_disconnect (input->parsebin, input->drained_sigid);
|
g_signal_handler_disconnect (input->parsebin, input->drained_sigid);
|
||||||
gst_element_set_state (input->parsebin, GST_STATE_NULL);
|
gst_element_set_state (input->parsebin, GST_STATE_NULL);
|
||||||
|
gst_bin_remove (GST_BIN (dbin), input->parsebin);
|
||||||
gst_clear_object (&input->parsebin);
|
gst_clear_object (&input->parsebin);
|
||||||
gst_clear_object (&input->parsebin_sink);
|
gst_clear_object (&input->parsebin_sink);
|
||||||
}
|
}
|
||||||
|
@ -1186,6 +1167,7 @@ reset_input (GstDecodebin3 * dbin, DecodebinInput * input)
|
||||||
gst_object_unref (idpad);
|
gst_object_unref (idpad);
|
||||||
remove_input_stream (dbin, stream);
|
remove_input_stream (dbin, stream);
|
||||||
gst_element_set_state (input->identity, GST_STATE_NULL);
|
gst_element_set_state (input->identity, GST_STATE_NULL);
|
||||||
|
gst_bin_remove (GST_BIN (dbin), input->identity);
|
||||||
gst_clear_object (&input->identity);
|
gst_clear_object (&input->identity);
|
||||||
}
|
}
|
||||||
if (input->collection)
|
if (input->collection)
|
||||||
|
|
Loading…
Reference in a new issue