[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAHC9VhQzToGSXyxdFKd=JrwKAkzhEsSx0z=sGsGLDZqL-bXJtA@mail.gmail.com>
Date: Mon, 13 Mar 2017 10:16:12 -0400
From: Paul Moore <paul@...l-moore.com>
To: Jessica Yu <jeyu@...hat.com>
Cc: Richard Guy Briggs <rgb@...hat.com>, linux-kernel@...r.kernel.org,
linux-audit@...hat.com
Subject: Re: [PATCH v2] audit: log module name on delete_module
On Sat, Mar 11, 2017 at 9:24 PM, Richard Guy Briggs <rgb@...hat.com> wrote:
> When a sysadmin wishes to monitor module unloading with a syscall rule such as:
> -a always,exit -F arch=x86_64 -S delete_module -F key=mod-unload
> the SYSCALL record doesn't tell us what module was requested for unloading.
>
> Use the new KERN_MODULE auxiliary record to record it.
> The SYSCALL record result code will list the return code.
>
> See: https://github.com/linux-audit/audit-kernel/issues/37
> https://github.com/linux-audit/audit-kernel/issues/7
> https://github.com/linux-audit/audit-kernel/wiki/RFE-Module-Load-Record-Format
>
> Signed-off-by: Richard Guy Briggs <rgb@...hat.com>
> ---
> kernel/module.c | 2 ++
> 1 files changed, 2 insertions(+), 0 deletions(-)
Jessica? If there are no objections to this patch on your side I'll
merge this into the audit/next tree.
> diff --git a/kernel/module.c b/kernel/module.c
> index 5432dbe..633f6da 100644
> --- a/kernel/module.c
> +++ b/kernel/module.c
> @@ -943,6 +943,8 @@ SYSCALL_DEFINE2(delete_module, const char __user *, name_user,
> return -EFAULT;
> name[MODULE_NAME_LEN-1] = '\0';
>
> + audit_log_kern_module(name);
> +
> if (mutex_lock_interruptible(&module_mutex) != 0)
> return -EINTR;
>
> --
> 1.7.1
>
> --
> Linux-audit mailing list
> Linux-audit@...hat.com
> https://www.redhat.com/mailman/listinfo/linux-audit
--
paul moore
www.paul-moore.com
Powered by blists - more mailing lists