On Fri, Jan 24, 2020 at 1:33 AM Vasily Averin <vvs@xxxxxxxxxxxxx> wrote: > > if seq_file .next fuction does not change position index, > read after some lseek can generate unexpected output. > > https://bugzilla.kernel.org/show_bug.cgi?id=206283 > Signed-off-by: Vasily Averin <vvs@xxxxxxxxxxxxx> > --- > security/selinux/selinuxfs.c | 1 + > 1 file changed, 1 insertion(+) It would be very nice if you could update this patch similar to what you did for the keys subsystem (archive link below). You included most of that information in your cover letter, but unfortunately cover letters don't make their way into the git log, only the individual commit descriptions. https://lore.kernel.org/linux-security-module/af9dcaa7-6e4f-281a-2bae-fb605cc55d2d@xxxxxxxxxxxxx Thank you. > diff --git a/security/selinux/selinuxfs.c b/security/selinux/selinuxfs.c > index ee94fa4..ad5530e 100644 > --- a/security/selinux/selinuxfs.c > +++ b/security/selinux/selinuxfs.c > @@ -1504,6 +1504,7 @@ static struct avc_cache_stats *sel_avc_get_stat_idx(loff_t *idx) > *idx = cpu + 1; > return &per_cpu(avc_cache_stats, cpu); > } > + (*idx)++; > return NULL; > } > > -- > 1.8.3.1 -- paul moore www.paul-moore.com
