lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<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

Powered by Openwall GNU/*/Linux Powered by OpenVZ