> On Tue, 2014-09-02 at 00:08 +0200, Peter Meerwald wrote: > > Debian/kFreeBSD 9.2 comes with sys/capability.h but it is not usable; work around it > > > > the patch does several things: > > * it makes the comment point to the correct bugtracker issue: https://bugs.freedesktop.org/show_bug.cgi?id=72580 > > * it handles Debian/kFreeBSD the same way as FreeBSD > > * it logs a warning that capabilities are actually NOT dropped > > > > daemon/caps.c: In function ?pa_drop_caps?: > > daemon/caps.c:93:2: error: #error "Don't know how to do capabilities on your system. Please send a patch." > > #error "Don't know how to do capabilities on your system. Please send a patch." > > ^ > > Makefile:9575: recipe for target 'daemon/pulseaudio-caps.o' failed > > > > Signed-off-by: Peter Meerwald <pmeerw at pmeerw.net> > > --- > > src/daemon/caps.c | 5 +++-- > > 1 file changed, 3 insertions(+), 2 deletions(-) > > > > diff --git a/src/daemon/caps.c b/src/daemon/caps.c > > index 669d6e2..f9d73c3 100644 > > --- a/src/daemon/caps.c > > +++ b/src/daemon/caps.c > > @@ -86,9 +86,10 @@ void pa_drop_caps(void) { > > pa_assert_se(cap_clear(caps) == 0); > > pa_assert_se(cap_set_proc(caps) == 0); > > pa_assert_se(cap_free(caps) == 0); > > -#elif defined(__FreeBSD__) > > +#elif defined(__FreeBSD__) || defined(__FreeBSD_kernel__) > > /* FreeBSD doesn't have this functionality, even though sys/capability.h is > > - * available. See https://bugs.freedesktop.org/show_bug.cgi?id=73967 */ > > + * available. See https://bugs.freedesktop.org/show_bug.cgi?id=72580 */ > > + pa_log_warn("FreeBSD cannot drop extra capabilities, implementation needed."); > > #else > > #error "Don't know how to do capabilities on your system. Please send a patch." > > #endif /* __linux__ */ > > Looks good to me. pushed -- Peter Meerwald +43-664-2444418 (mobile)
