lkml.org 
[lkml]   [2022]   [Oct]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 6.0 247/862] audit: free audit_proctitle only on task exit
    Date
    From: Richard Guy Briggs <rgb@redhat.com>

    [ Upstream commit c3f3ea8af44d0c5fba79fe8b198087342d0c7e04 ]

    Since audit_proctitle is generated at syscall exit time, its value is
    used immediately and cached for the next syscall. Since this is the
    case, then only clear it at task exit time. Otherwise, there is no
    point in caching the value OR bearing the overhead of regenerating it.

    Fixes: 12c5e81d3fd0 ("audit: prepare audit_context for use in calling contexts beyond syscalls")
    Signed-off-by: Richard Guy Briggs <rgb@redhat.com>
    Signed-off-by: Paul Moore <paul@paul-moore.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    kernel/auditsc.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    diff --git a/kernel/auditsc.c b/kernel/auditsc.c
    index 0ee09447ad04..63a6fe99aa3a 100644
    --- a/kernel/auditsc.c
    +++ b/kernel/auditsc.c
    @@ -1016,7 +1016,6 @@ static void audit_reset_context(struct audit_context *ctx)
    WARN_ON(!list_empty(&ctx->killed_trees));
    audit_free_module(ctx);
    ctx->fds[0] = -1;
    - audit_proctitle_free(ctx);
    ctx->type = 0; /* reset last for audit_free_*() */
    }

    @@ -1077,6 +1076,7 @@ static inline void audit_free_context(struct audit_context *context)
    {
    /* resetting is extra work, but it is likely just noise */
    audit_reset_context(context);
    + audit_proctitle_free(context);
    free_tree_refs(context);
    kfree(context->filterkey);
    kfree(context);
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-10-19 15:58    [W:2.713 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site