[pulseaudio-discuss] [PATCH 2/2] client-conf: Don't create multiple cookie files
tanu.kaskinen at linux.intel.com
Fri Mar 28 06:06:57 PDT 2014
On Wed, 2014-03-19 at 12:19 +0200, Tanu Kaskinen wrote:
> The old code loaded cookies at the time of loading the client
> configuration, which could lead to creation of multiple cookie files.
> For example, when pa_client_conf_load() was called, the default cookie
> file was created, and then if PULSE_COOKIE was set,
> pa_client_conf_env() would create another cookie file.
> This patch moves the loading of the cookie to a separate function,
> which pa_context calls just before needing the cookie, so the cookie
> won't be loaded from the default file if PULSE_COOKIE is set. This
> patch also splits the single cookie and cookie_file fields in
> pa_client_conf into multiple fields, one for each possible cookie
> source. That change allows falling back to another cookie source if
> the primary source doesn't work.
> BugLink: https://bugs.freedesktop.org/show_bug.cgi?id=75006
> src/pulse/client-conf-x11.c | 4 +-
> src/pulse/client-conf.c | 148 +++++++++++++++++++++++---------------------
> src/pulse/client-conf.h | 29 +++++----
> src/pulse/context.c | 11 ++--
> 4 files changed, 104 insertions(+), 88 deletions(-)
I pushed these now.
More information about the pulseaudio-discuss