mirrors/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire.git synced 2025-11-01 22:58:50 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

audioconvert: handle recursive Props set_param

Browse source 
When the node receives a set_param Props, it calls the follower
set_param implementation. If that one calls set_param again on the
adapter, discards the original set_param.

This makes it possible for the follower to intercept the Props param and
set a modified version on the converter. This can be used to intercept
volume changes.
This commit is contained in:
Wim Taymans 2023-04-17 16:12:35 +02:00
parent dad87fb3a8
commit bf1b3bb157
1 changed files with 7 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

10
spa/plugins/audioconvert/audioadapter.c
View file

@ -51,6 +51,7 @@ struct impl {
uint64_t follower_flags;
struct spa_audio_info follower_current_format;
struct spa_audio_info default_format;
int in_set_param;
struct spa_handle *hnd_convert;
struct spa_node *convert;
@ -675,13 +676,16 @@ static int impl_node_set_param(void *object, uint32_t id, uint32_t flags,
}
case SPA_PARAM_Props:
if (this->target != this->follower)
res = spa_node_set_param(this->target, id, flags, param);
res2 = spa_node_set_param(this->follower, id, flags, param);
{
int in_set_param = ++this->in_set_param;
res = spa_node_set_param(this->follower, id, flags, param);
if (this->target != this->follower && this->in_set_param == in_set_param)
res2 = spa_node_set_param(this->target, id, flags, param);
if (res < 0 && res2 < 0)
return res;
res = 0;
break;
}
case SPA_PARAM_ProcessLatency:
res = spa_node_set_param(this->follower, id, flags, param);
break;