All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve Grubb <sgrubb@redhat.com>
To: Arnd Bergmann <arnd@arndb.de>, Ricardo Robaina <rrobaina@redhat.com>
Cc: audit@vger.kernel.org, linux-kernel@vger.kernel.org,
	Linux-Arch <linux-arch@vger.kernel.org>,
	Paul Moore <paul@paul-moore.com>, Eric Paris <eparis@redhat.com>
Subject: Re: [PATCH] audit: add missing syscalls to PERM class tables
Date: Wed, 10 Jun 2026 14:13:44 -0400	[thread overview]
Message-ID: <2021591.7Z3S40VBb9@x2> (raw)
In-Reply-To: <CAABTaaBBVP6eY8D+a1KTgfZ3x8v3egnKTiQLRQfXPiOhpmQXjg@mail.gmail.com>

On Wednesday, June 10, 2026 1:40:47 PM Eastern Daylight Time Ricardo Robaina 
wrote:
> On Wed, Jun 10, 2026 at 2:05 PM Arnd Bergmann <arnd@arndb.de> wrote:
> > On Wed, Jun 10, 2026, at 18:47, Ricardo Robaina wrote:
> > > diff --git a/include/asm-generic/audit_read.h
> > > b/include/asm-generic/audit_read.h
> > > index fb9991f53fb6..8feebc5b4c50 100644
> > > --- a/include/asm-generic/audit_read.h
> > > +++ b/include/asm-generic/audit_read.h
> > > @@ -18,3 +21,19 @@ __NR_fgetxattr,
> > > 
> > >  #ifdef __NR_readlinkat
> > >  __NR_readlinkat,
> > >  #endif
> > > 
> > > +#ifdef __NR_file_getattr
> > > +__NR_file_getattr,
> > > +#endif
> > > +#ifdef __NR_stat
> > > +__NR_stat,
> > > +#endif
> > > +#ifdef __NR_lstat
> > > +__NR_lstat,
> > > +#endif
> > > +#ifdef __NR_fstat
> > > +__NR_fstat,
> > > +#endif
> > > +#ifdef __NR_newfstatat
> > > +__NR_newfstatat,
> > > +#endif
> > > +__NR_statx,
> > 
> > There are additional variants of 'stat' that I think you need
> > to cover here:
> > 
> > scripts/syscall.tbl:79  stat64  fstatat64                      
> > sys_fstatat64 scripts/syscall.tbl:80  stat64  fstat64                   
> >      sys_fstat64 arch/x86/entry/syscalls/syscall_32.tbl:18       i386   
> > oldstat                 sys_stat
> > arch/x86/entry/syscalls/syscall_32.tbl:28       i386    oldfstat        
> >        sys_fstat arch/x86/entry/syscalls/syscall_32.tbl:84       i386   
> > oldlstat                sys_lstat
> > arch/x86/entry/syscalls/syscall_32.tbl:195      i386    stat64          
> >        sys_stat64                      compat_sys_ia32_stat64
> > arch/x86/entry/syscalls/syscall_32.tbl:196      i386    lstat64         
> >        sys_lstat64                     compat_sys_ia32_lstat64
> > arch/x86/entry/syscalls/syscall_32.tbl:197      i386    fstat64         
> >        sys_fstat64                     compat_sys_ia32_fstat64
> > arch/x86/entry/syscalls/syscall_32.tbl:300      i386    fstatat64       
> >        sys_fstatat64                   compat_sys_ia32_fstatat64
> > arch/alpha/kernel/syscalls/syscall.tbl:224      common  osf_stat        
> >                sys_osf_stat arch/alpha/kernel/syscalls/syscall.tbl:225  
> >    common  osf_lstat                       sys_osf_lstat
> > arch/alpha/kernel/syscalls/syscall.tbl:226      common  osf_fstat       
> >                sys_osf_fstat
> Hi Arnd,
> 
> Thanks for reviewing this patch! You're right, it seems all these stat
> variants should be added as well. Steve and Paul, correct me if I'm
> wrong here, please.

Alpha is unsupported. Those are True64 compatibility syscalls. You can 
include it and #ifdef will filter it everywhere else.  But, yeah. I guess the 
rest are ok. I don't pay much attention to the 32 bit arches.

-Steve



  reply	other threads:[~2026-06-10 18:13 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-06-10 16:47 [PATCH] audit: add missing syscalls to PERM class tables Ricardo Robaina
2026-06-10 17:05 ` Arnd Bergmann
2026-06-10 17:40   ` Ricardo Robaina
2026-06-10 18:13     ` Steve Grubb [this message]
2026-06-10 18:54       ` Ricardo Robaina
2026-06-10 19:39       ` Arnd Bergmann
2026-06-10 19:53         ` Ricardo Robaina

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=2021591.7Z3S40VBb9@x2 \
    --to=sgrubb@redhat.com \
    --cc=arnd@arndb.de \
    --cc=audit@vger.kernel.org \
    --cc=eparis@redhat.com \
    --cc=linux-arch@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=paul@paul-moore.com \
    --cc=rrobaina@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.