linux-kernel - Re: [PATCH v3] selftests/seccomp: Fix indentation and rebase error logging patch

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] [day] [month] [year] [list]

Message-ID: <CAGZaOhcmHt76q=ABQ-NJvj9tuedCeonEFGP8SvnhUp9=+T5tPQ@mail.gmail.com>
Date: Tue, 25 Nov 2025 09:34:18 +0530
From: Sameeksha Sankpal <sameekshasankpal@...il.com>
To: Shuah Khan <skhan@...uxfoundation.org>
Cc: kees@...nel.org, luto@...capital.net, wad@...omium.org, shuah@...nel.org, 
	linux-kernel@...r.kernel.org, linux-kselftest@...r.kernel.org
Subject: Re: [PATCH v3] selftests/seccomp: Fix indentation and rebase error
 logging patch

Hi Shuah,

Thanks for pointing that out. Apologies for missing the mailing lists earlier.
Resending this follow-up with the correct CC list and in plain text format.

Please let me know if there’s anything else I should improve in this patch.
I’m happy to resend it as v4 if needed.

Thanks,
Sameeksha


On Mon, 24 Nov 2025 at 23:59, Shuah Khan <skhan@...uxfoundation.org> wrote:
>
> On 11/21/25 23:21, Sameeksha Sankpal wrote:
> > Hi,
> > Just following up on this patch.
> > It’s been a few months, so I wanted to check if there is anything else I
> > should address or improve to move it forward.
>
> I see that you didn't cc any mailing list on this email? Please keep
> everybody in the loop when you send responses.
>
> >
> > Thanks,
> > Sameeksha Sankpal
> >
> > On Fri, 30 May 2025 at 04:25, Sameeksha Sankpal <sameekshasankpal@...il.com>
> > wrote:
> >
> >> Rebase the error logging enhancement for get_proc_stat() against the
> >> upstream seccomp tree with proper indentation formatting.
> >>
> >> Suggested-by: Kees Cook <kees@...nel.org>
> >> Signed-off-by: Sameeksha Sankpal <sameekshasankpal@...il.com>
> >> ---
> >> v1 -> v2:
> >> - Used TH_LOG instead of printf for error logging
> >> - Moved variable declaration to the top of the function
> >> - Applied review suggestion by Kees Cook
> >>
> >> v2 -> v3:
> >> - Rebased against upstream seccomp tree (was previously against v1)
> >> - Fixed indentation to use tabs instead of spaces
> >> - Used scripts/checkpatch.pl to check the patch for common errors
> >> - Removed the blank line beforeS S-o-b added in v2
> >>
> >>   tools/testing/selftests/seccomp/seccomp_bpf.c | 5 +++++
> >>   1 file changed, 5 insertions(+)
> >>
> >> diff --git a/tools/testing/selftests/seccomp/seccomp_bpf.c
> >> b/tools/testing/selftests/seccomp/seccomp_bpf.c
> >> index 61acbd45ffaa..dbd7e705a2af 100644
> >> --- a/tools/testing/selftests/seccomp/seccomp_bpf.c
> >> +++ b/tools/testing/selftests/seccomp/seccomp_bpf.c
> >> @@ -4508,9 +4508,14 @@ static char get_proc_stat(struct __test_metadata
> >> *_metadata, pid_t pid)
> >>          char proc_path[100] = {0};
> >>          char status;
> >>          char *line;
> >> +       int rc;
> >>
> >>          snprintf(proc_path, sizeof(proc_path), "/proc/%d/stat", pid);
> >>          ASSERT_EQ(get_nth(_metadata, proc_path, 3, &line), 1);
> >> +       rc = get_nth(_metadata, proc_path, 3, &line);
> >> +       ASSERT_EQ(rc, 1) {
> >> +               TH_LOG("user_notification_fifo: failed to read stat for
> >> PID %d (rc=%d)", pid, rc);
> >> +       }
> >>
> >>          status = *line;
> >>          free(line);
> >> --
> >> 2.43.0
> >>
> >>
> >
> thanks,
> -- Shuah