# Re: \[PATCH v2 17/27\] selftests/landlock: Fix SIGURG handler setup in scoped\_signal\_test

- From: sashiko-bot@kernel.org
- Date: 2026-11-04 21:15:01 
- List: linux-security-module
- Page: https://ratatoskr.run/linux-security-module/2026/11/17864695
- Thread: https://ratatoskr.run/linux-security-module/2026/11/17864489/t.md

```text
> Drop SA_SIGINFO, and rename the function to setup_sigurg_handler()
> without the parameter.
> 
> * Drop the SA_SIGINFO flag: This flag implies that we use the
>   three-argument sa_sigaction signal handlers, which is not the case
>   here. (c.f. sigaction(2))
> * Remove the "int signal" argument and rename the function:
>   The function was already specific to SIGURG and this is better
>   communicated through a different name.
> 
> Signed-off-by: Günther Noack <redacted>

Sashiko has reviewed this patch and found no issues.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/20261104-selftests-refactor-v2-0-c52f23f7ce33@gmail.com?part=17

```