[pulseaudio-discuss] [PATCH] daemon-conf: Don't make log files executable.

Colin Guthrie gmane at colin.guthr.ie
Sat Aug 13 04:57:37 PDT 2011


'Twas brillig, and Tanu Kaskinen at 13/08/11 13:35 did gyre and gimble:
> ---
>  src/daemon/daemon-conf.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/src/daemon/daemon-conf.c b/src/daemon/daemon-conf.c
> index 1f70b09..c4ee544 100644
> --- a/src/daemon/daemon-conf.c
> +++ b/src/daemon/daemon-conf.c
> @@ -194,7 +194,7 @@ int pa_daemon_conf_set_log_target(pa_daemon_conf *c, const char *string) {
>          pa_strlcpy(file_path, string + 5, sizeof(file_path));
>  
>          /* Open target file with user rights */
> -        if ((log_fd = open(file_path, O_RDWR|O_TRUNC|O_CREAT, S_IRWXU)) >= 0) {
> +        if ((log_fd = open(file_path, O_RDWR|O_TRUNC|O_CREAT, S_IRUSR | S_IWUSR)) >= 0) {
>               c->auto_log_target = 0;
>               c->log_target = PA_LOG_FD;
>               pa_log_set_fd(log_fd);

In my tree now. Thanks

Col

-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
  Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
  Mageia Contributor [http://www.mageia.org/]
  PulseAudio Hacker [http://www.pulseaudio.org/]
  Trac Hacker [http://trac.edgewall.org/]



More information about the pulseaudio-discuss mailing list