Usually autospawning for root is a bad idea, since it can easily interfere with other users' PulseAudio instances, but in embedded environments where only root exists, autospawning is fine. --- man/pulse-client.conf.5.xml.in | 9 +++++++++ src/pulse/client-conf.c | 1 + src/pulse/client-conf.h | 1 + src/pulse/client.conf.in | 1 + src/pulse/context.c | 2 +- 5 files changed, 13 insertions(+), 1 deletion(-) diff --git a/man/pulse-client.conf.5.xml.in b/man/pulse-client.conf.5.xml.in index 1002dbe..0058490 100644 --- a/man/pulse-client.conf.5.xml.in +++ b/man/pulse-client.conf.5.xml.in @@ -71,6 +71,15 @@ License along with PulseAudio; if not, see <http://www.gnu.org/licenses/>. </option> <option> + <p><opt>allow-autospawn-for-root=</opt> Allow autospawning also for root. + Takes a boolean value, defaults to <opt>no</opt>. If the <opt>autospawn + </opt> option is disabled, this option has no effect. Autospawning for + root is disabled by default, because running PulseAudio as root will + interfere with regular users' PulseAudio instances. This option should be + enabled only in environments where there are no regular users at all.</p> + </option> + + <option> <p><opt>daemon-binary=</opt> Path to the PulseAudio daemon to run when autospawning. Defaults to a path configured at compile time.</p> diff --git a/src/pulse/client-conf.c b/src/pulse/client-conf.c index 83331f8..0474583 100644 --- a/src/pulse/client-conf.c +++ b/src/pulse/client-conf.c @@ -138,6 +138,7 @@ void pa_client_conf_load(pa_client_conf *c, bool load_from_x11, bool load_from_e { "default-server", pa_config_parse_string, &c->default_server, NULL }, { "default-dbus-server", pa_config_parse_string, &c->default_dbus_server, NULL }, { "autospawn", pa_config_parse_bool, &c->autospawn, NULL }, + { "allow-autospawn-for-root", pa_config_parse_bool, &c->allow_autospawn_for_root, NULL }, { "cookie-file", pa_config_parse_string, &c->cookie_file_from_client_conf, NULL }, { "disable-shm", pa_config_parse_bool, &c->disable_shm, NULL }, { "enable-shm", pa_config_parse_not_bool, &c->disable_shm, NULL }, diff --git a/src/pulse/client-conf.h b/src/pulse/client-conf.h index eac705a..131393a 100644 --- a/src/pulse/client-conf.h +++ b/src/pulse/client-conf.h @@ -38,6 +38,7 @@ typedef struct pa_client_conf { char *cookie_file_from_application; char *cookie_file_from_client_conf; bool autospawn, disable_shm, auto_connect_localhost, auto_connect_display; + bool allow_autospawn_for_root; size_t shm_size; } pa_client_conf; diff --git a/src/pulse/client.conf.in b/src/pulse/client.conf.in index 26b7790..69830ef 100644 --- a/src/pulse/client.conf.in +++ b/src/pulse/client.conf.in @@ -23,6 +23,7 @@ ; default-dbus-server = ; autospawn = yes +; allow-autospawn-for-root = no ; daemon-binary = @PA_BINARY@ ; extra-arguments = --log-target=syslog diff --git a/src/pulse/context.c b/src/pulse/context.c index 738ea84..c723446 100644 --- a/src/pulse/context.c +++ b/src/pulse/context.c @@ -981,7 +981,7 @@ int pa_context_connect( if (!(flags & PA_CONTEXT_NOAUTOSPAWN) && c->conf->autospawn) { #ifdef HAVE_GETUID - if (getuid() == 0) + if (!c->conf->allow_autospawn_for_root && getuid() == 0) pa_log_debug("Not doing autospawn since we are root."); else { c->do_autospawn = true; -- 1.9.3
