mirrors/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire.git synced 2026-07-05 00:06:16 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge branch 'fix_filter_graph_crash' into 'master'

Browse source 
Fix NULL filter-graph handle crash during graph rebuild

See merge request pipewire/pipewire!2883
This commit is contained in:
Torkel Niklasson 2026-06-29 08:17:11 +00:00
commit 88e0d08987
1 changed files with 16 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

16
spa/plugins/audioconvert/audioconvert.c
View file

@ -1368,6 +1368,8 @@ static int ensure_tmp(struct impl *this)
}
static void sync_filter_graph(struct impl *impl);
static int setup_filter_graphs(struct impl *impl, bool force)
{
int res;
@ -1382,6 +1384,20 @@ static int setup_filter_graphs(struct impl *impl, bool force)
position = in->format.info.raw.position;
impl->maxports = SPA_MAX(in->format.info.raw.channels, out->format.info.raw.channels);
if (force) {
/* A forced setup deactivates and re-instantiates each graph below,
* which frees and recreates the underlying plugin handles. Pull the
* graphs out of the data-loop's view first (under the loop lock, via
* sync_filter_graph) so the RT thread cannot run a graph while its
* handles are NULL during the rebuild. They are republished by the
* sync_filter_graph() that follows setup. */
spa_list_for_each(g, &impl->active_graphs, link) {
if (!g->removing)
g->setup = false;
}
sync_filter_graph(impl);
}
spa_list_for_each_safe(g, t, &impl->active_graphs, link) {
if (g->removing)
continue;