Attachment 106308 Details for Bug 83856 – 0002-Throw-a-warning-if-we-cannot-open-the-audit-socket-a.patch

[patch] 0002-Throw-a-warning-if-we-cannot-open-the-audit-socket-a.patch

0002-Throw-a-warning-if-we-cannot-open-the-audit-socket-a.patch (text/plain), 1.07 KB, created by Laurent Bigonville on 2014-09-15 09:44:25 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Laurent Bigonville

Created: 2014-09-15 09:44:25 UTC

Size: 1.07 KB

patch

obsolete

>From 6cbca32b17568ef3f4da95df90bd45666e10b9a5 Mon Sep 17 00:00:00 2001
>From: Laurent Bigonville <bigon@bigon.be>
>Date: Mon, 15 Sep 2014 11:25:39 +0200
>Subject: [PATCH 2/2] Throw a warning if we cannot open the audit socket as a
> session bus
>
>This will effectively print a warning when failing to open the audit
>socket running as a session bus.
>
>The call to audit_open() should succeed even if the dbus-daemon doesn't
>have the CAP_AUDIT_WRITE capability.
>
>https://bugs.freedesktop.org/show_bug.cgi?id=83856
>---
> bus/selinux.c | 3 ---
> 1 file changed, 3 deletions(-)
>
>diff --git a/bus/selinux.c b/bus/selinux.c
>index 99994ca..56416b4 100644
>--- a/bus/selinux.c
>+++ b/bus/selinux.c
>@@ -130,9 +130,6 @@ bus_selinux_audit_init(void)
>       /* If kernel doesn't support audit, bail out */
>       if (errno == EINVAL || errno == EPROTONOSUPPORT || errno == EAFNOSUPPORT)
>         return;
>-      /* If user bus, bail out */
>-      if (errno == EPERM && getuid() != 0)
>-        return;
>       _dbus_warn ("Failed opening connection to the audit subsystem");
>     }
> #endif /* HAVE_LIBAUDIT */
>-- 
>2.1.0
>

Actions: View | Splinter Review

Attachments on bug 83856: 106306 | 106308 | 116435 | 116436 | 116510 | 116511

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.