[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