Re: [RFC PATCH v7 07/16] ipe: add auditing support
Hello,
On Friday, October 15, 2021 3:25:47 PM EDT Deven Bowers wrote:
On 10/13/2021 1:02 PM, Steve Grubb wrote:
> On Wednesday, October 13, 2021 3:06:26 PM EDT
> deven.desai(a)linux.microsoft.com>
> wrote:
>> Users of IPE require a way to identify when and why an operation fails,
>> allowing them to both respond to violations of policy and be notified
>> of potentially malicious actions on their systens with respect to IPE
>> itself.
>
> Would you mind sending examples of audit events so that we can see what
> the end result is? Some people add them to the commit text. But we still
> need to see what they look like.
Sure, sorry. I’ll add them to the commit description (and the documentation
patch at the end) for v8 – In the interest of asynchronous feedback, I’ve
copied the relevant examples:
Thanks for sending these. This helps.
AUDIT1420 IPE ctx_pid=229 ctx_op=EXECUTE ctx_hook=MMAP ctx_enforce=0
ctx_comm="grep" ctx_pathname="/usr/lib/libc-2.23.so"
ctx_ino=532 ctx_dev=vda rule="DEFAULT op=EXECUTE action=DENY"
Question...why do all of these have a ctx_ prefix? Is it possible to trigger
an audit context so that the audit machinery collects all of this stuff in
it's own way? Which means you could drop everything execept op, hook,
enforce, rule, and action.
We also have a field dictionary here:
https://github.com/linux-audit/audit-documentation/blob/main/specs/fields/
field-dictionary.csv
which names the known fields and how they should be formatted. If there is a
collision where they are something else and cannot be in the same format,
then we make a new name and hopefully update the dictionary. For example, if
you are collecting a process id, use pid and not ctx_pid so that it matches a
known definition.
Also, I don't thnk these events can stand on their own. Who did this action?
You have the pid, but no uid, auid, or session_id.
Hope this helps...
-Steve
AUDIT1420 IPE ctx_pid=229 ctx_op=EXECUTE ctx_hook=MMAP ctx_enforce=0
ctx_comm="grep" ctx_pathname="/usr/lib/libc-2.23.so"
ctx_ino=532 ctx_dev=vda rule="DEFAULT action=DENY"
AUDIT1420 IPE ctx_pid=253 ctx_op=EXECUTE ctx_hook=MMAP ctx_enforce=1
ctx_comm="anon" rule="DEFAULT op=EXECUTE action=DENY"
These three audit records represent various types of results after
evaluating
the trust of a resource. The first two differ in the rule that was
matched in
IPE's policy, the first being an operation-specific default, the second
being
a global default. The third is an example of what is audited when anonymous
memory is blocked (as there is no way to verify the trust of an anonymous
page).
The remaining three events, AUDIT_TRUST_POLICY_LOAD (1421),
AUDIT_TRUST_POLICY_ACTIVATE (1422), and AUDIT_TRUST_STATUS (1423) have this
form:
AUDIT1421 IPE policy_name="my-policy" policy_version=0.0.0
<hash_alg_name>=<hash>
AUDIT1422 IPE policy_name="my-policy" policy_version=0.0.0
<hash_alg_name>=<hash>
AUDIT1423 IPE enforce=1
The 1421 (AUDIT_TRUST_POLICY_LOAD) event represents a new policy was loaded
into the kernel, but not is not marked as the policy to enforce. The
The 1422 (AUDIT_TRUST_POLICY_ACTIVATE) event represents a policy that was
already loaded was made the enforcing policy.
The 1423 (AUDIT_TRUST_STATUS) event represents a switch between
permissive and
enforce, it is added in 08/16 (the following patch)