[09/26] Linux: sigaltstack syscall number is always available

Message ID a19e646fd70d38f688a41daf15650f56d66cd7d9.1581279333.git.fweimer@redhat.com
State New
Headers show
Series
  • Linux cleanups enabled by built-in system call tables
Related show

Commit Message

Florian Weimer Feb. 9, 2020, 8:20 p.m.
Due to the built-in tables, __NR_sigaltstack is always defined.
---
 sysdeps/unix/sysv/linux/sigstack.c | 4 ----
 1 file changed, 4 deletions(-)

-- 
2.24.1

Comments

Adhemerval Zanella Feb. 27, 2020, 11:06 p.m. | #1
On 09/02/2020 17:20, Florian Weimer wrote:
> Due to the built-in tables, __NR_sigaltstack is always defined.


LGTM, thanks. 

Reviewed-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>


> ---

>  sysdeps/unix/sysv/linux/sigstack.c | 4 ----

>  1 file changed, 4 deletions(-)

> 

> diff --git a/sysdeps/unix/sysv/linux/sigstack.c b/sysdeps/unix/sysv/linux/sigstack.c

> index 2f75e2643a..32a378bd26 100644

> --- a/sysdeps/unix/sysv/linux/sigstack.c

> +++ b/sysdeps/unix/sysv/linux/sigstack.c

> @@ -22,7 +22,6 @@

>  #include <sys/syscall.h>

>  

>  

> -#ifdef __NR_sigaltstack

>  int

>  sigstack (struct sigstack *ss, struct sigstack *oss)

>  {

> @@ -61,6 +60,3 @@ sigstack (struct sigstack *ss, struct sigstack *oss)

>  }

>  

>  link_warning (sigstack, "the `sigstack' function is dangerous.  `sigaltstack' should be used instead.")

> -#else

> -# include <signal/sigstack.c>

> -#endif

>

Patch

diff --git a/sysdeps/unix/sysv/linux/sigstack.c b/sysdeps/unix/sysv/linux/sigstack.c
index 2f75e2643a..32a378bd26 100644
--- a/sysdeps/unix/sysv/linux/sigstack.c
+++ b/sysdeps/unix/sysv/linux/sigstack.c
@@ -22,7 +22,6 @@ 
 #include <sys/syscall.h>
 
 
-#ifdef __NR_sigaltstack
 int
 sigstack (struct sigstack *ss, struct sigstack *oss)
 {
@@ -61,6 +60,3 @@  sigstack (struct sigstack *ss, struct sigstack *oss)
 }
 
 link_warning (sigstack, "the `sigstack' function is dangerous.  `sigaltstack' should be used instead.")
-#else
-# include <signal/sigstack.c>
-#endif