selinux.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Smalley <sds@tycho.nsa.gov>
To: Ted Toth <txtoth@gmail.com>
Cc: selinux@vger.kernel.org
Subject: Re: strange tclass in AVCs
Date: Wed, 18 Sep 2019 10:17:42 -0400	[thread overview]
Message-ID: <b2c71d86-c574-43d6-2fcf-17a06078c8df@tycho.nsa.gov> (raw)
In-Reply-To: <CAFPpqQGfAifnShcWTMs1pJe1m7W_4wM8AUqFsvj4Zts5-9QiCQ@mail.gmail.com>

On 9/18/19 10:03 AM, Ted Toth wrote:
> On Wed, Sep 18, 2019 at 8:53 AM Stephen Smalley <sds@tycho.nsa.gov> wrote:
>>
>> On 9/18/19 9:35 AM, Ted Toth wrote:
>>> I'm seeing things like tclass=context#012 in some AVCs what is this telling me?
>>
>> Just a guess here, but octal 012 is '\n' aka a newline character, and
>> libselinux/src/avc.c:avc_audit() appends a "\n" at the end of the buffer
>> before calling avc_log() to log the entire string.  avc_log() will call
>> the logging callback, and dbusd does define one, which calls
>> audit_log_user_avc_message().  Maybe audit_log_user_avc_message() is
>> escaping the newline character in its output as well as appending
>> additional data.
>>
>> I'm a little unclear though on why dbusd is checking a context contains
>> permission?
> 
> These appear to only occur when systemd is starting the dbus daemon
> and they end up in /var/log/messages not /var/log/audit/audit.log as
> I'd expect.

Sounds like auditd isn't operational at that point and therefore the 
output just goes to syslog.

Arguably avc_audit() shouldn't be adding a newline at all and that 
should be handled by the logging callback (or default_selinux_log if no 
callback is set).  But it has been this way forever, so that would no 
doubt break some users.  Legacy of when this was a printk/printf.





  parent reply	other threads:[~2019-09-18 14:17 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-18 13:35 strange tclass in AVCs Ted Toth
2019-09-18 13:46 ` Dominick Grift
2019-09-18 13:49   ` Dominick Grift
2019-09-18 13:51     ` Ted Toth
2019-09-18 13:53 ` Stephen Smalley
2019-09-18 14:03   ` Ted Toth
2019-09-18 14:11     ` Ted Toth
2019-09-18 14:17     ` Stephen Smalley [this message]
2019-09-19 13:02       ` Ted Toth
2019-09-19 17:03         ` Stephen Smalley
2019-09-19 17:19           ` Ted Toth
2019-09-19 17:24             ` Stephen Smalley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=b2c71d86-c574-43d6-2fcf-17a06078c8df@tycho.nsa.gov \
    --to=sds@tycho.nsa.gov \
    --cc=selinux@vger.kernel.org \
    --cc=txtoth@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).