<div dir="ltr"><div>Thx for review and changes, looks better indeed. I also confirm it still works.<br></div><div>Cheers, Julien<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On 19 October 2015 at 04:23, Arun Raghavan <span dir="ltr"><<a href="mailto:arun@accosted.net" target="_blank">arun@accosted.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sat, 2015-10-10 at 20:11 +0100, Julien Isorce wrote:<br>
> fchmod is denied in chromium sandbox.<br>
> ---<br>
>  src/pulsecore/core-util.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/src/pulsecore/core-util.c b/src/pulsecore/core-util.c<br>
> index 278ad63..f694347 100644<br>
> --- a/src/pulsecore/core-util.c<br>
> +++ b/src/pulsecore/core-util.c<br>
> @@ -343,7 +343,7 @@ again:<br>
>  #endif<br>
><br>
>  #ifdef HAVE_FCHMOD<br>
> -    if (fchmod(fd, m) < 0) {<br>
> +    if ((st.st_mode & 0x0FFF) != m && fchmod(fd, m) < 0) {<br>
>          pa_assert_se(pa_close(fd) >= 0);<br>
>          goto fail;<br>
>      };<br>
<br>
</span>I'm pushing this out with some minor changes. The commit log is made to<br>
be more consistent with previous commits.<br>
<br>
I also switched 0xFFFF to 07777 since that's the more common convention<br>
with mode-related variables (this really is a nitpick, though!)<br>
<br>
-- Arun<br>
_______________________________________________<br>
pulseaudio-discuss mailing list<br>
<a href="mailto:pulseaudio-discuss@lists.freedesktop.org">pulseaudio-discuss@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/pulseaudio-discuss</a><br>
</blockquote></div><br></div>