[LTP] [PATCH] sigaltstack01: Fix check about alternative stack

Petr Vorel pvorel@suse.cz
Mon Aug 18 15:17:33 CEST 2025


Hi Liheng,

> Hi all,

> This check seems should be (alt_stk < sigstk.ss_sp) || (alt_stk > (sigstk.ss_sp + SIGSTKSZ)), not &&

You're right. IMHO the error goes down to the original addition in
865695bbc89088b9526ea9045410e5afb70a985c

Out of curiosity, did you find a system where it should fail but it didn't?

Reviewed-by: Petr Vorel <pvorel@suse.cz>

> Kind regards,
> Liheng Chen

> Signed-off-by: Liheng Chen <791960492@qq.com>
> ---
>  testcases/kernel/syscalls/sigaltstack/sigaltstack01.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

> diff --git a/testcases/kernel/syscalls/sigaltstack/sigaltstack01.c b/testcases/kernel/syscalls/sigaltstack/sigaltstack01.c
> index 9a2e3a440..147659467 100644
> --- a/testcases/kernel/syscalls/sigaltstack/sigaltstack01.c
> +++ b/testcases/kernel/syscalls/sigaltstack/sigaltstack01.c
> @@ -142,7 +142,7 @@ int main(int ac, char **av)
>  			 * Check that main_stk is outside the
>  			 * alternate stk boundaries.
>  			 */
> -			if ((alt_stk < sigstk.ss_sp) &&
> +			if ((alt_stk < sigstk.ss_sp) ||
>  			    (alt_stk > (sigstk.ss_sp + SIGSTKSZ))) {
>  				tst_resm(TFAIL,
>  					 "alt. stack is not within the "


More information about the ltp mailing list