Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
sysconf: Add _SC_MINSIGSTKSZ/_SC_SIGSTKSZ [BZ #20305]
Add _SC_MINSIGSTKSZ for the minimum signal stack size derived from AT_MINSIGSTKSZ, which is the minimum number of bytes of free stack space required in order to gurantee successful, non-nested handling of a single signal whose handler is an empty function, and _SC_SIGSTKSZ which is the suggested minimum number of bytes of stack space required for a signal stack. If AT_MINSIGSTKSZ isn't available, sysconf (_SC_MINSIGSTKSZ) returns MINSIGSTKSZ. On Linux/x86 with XSAVE, the signal frame used by kernel is composed of the following areas and laid out as: ------------------------------ | alignment padding | ------------------------------ | xsave buffer | ------------------------------ | fsave header (32-bit only) | ------------------------------ | siginfo + ucontext | ------------------------------ Compute AT_MINSIGSTKSZ value as size of xsave buffer + size of fsave header (32-bit only) + size of siginfo and ucontext + alignment padding. If _SC_SIGSTKSZ_SOURCE or _GNU_SOURCE are defined, MINSIGSTKSZ and SIGSTKSZ are redefined as /* Default stack size for a signal handler: sysconf (SC_SIGSTKSZ). */ # undef SIGSTKSZ # define SIGSTKSZ sysconf (_SC_SIGSTKSZ) /* Minimum stack size for a signal handler: SIGSTKSZ. */ # undef MINSIGSTKSZ # define MINSIGSTKSZ SIGSTKSZ Compilation will fail if the source assumes constant MINSIGSTKSZ or SIGSTKSZ. The reason for not simply increasing the kernel's MINSIGSTKSZ #define (apart from the fact that it is rarely used, due to glibc's shadowing definitions) was that userspace binaries will have baked in the old value of the constant and may be making assumptions about it. For example, the type (char [MINSIGSTKSZ]) changes if this #define changes. This could be a problem if an newly built library tries to memcpy() or dump such an object defined by and old binary. Bounds-checking and the stack sizes passed to things like sigaltstack() and makecontext() could similarly go wrong.
- Loading branch information
Showing
23 changed files
with
416 additions
and
3 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,21 @@ | ||
/* Definition of MINSIGSTKSZ. Generic version. | ||
Copyright (C) 2020 Free Software Foundation, Inc. | ||
This file is part of the GNU C Library. | ||
The GNU C Library is free software; you can redistribute it and/or | ||
modify it under the terms of the GNU Lesser General Public | ||
License as published by the Free Software Foundation; either | ||
version 2.1 of the License, or (at your option) any later version. | ||
The GNU C Library is distributed in the hope that it will be useful, | ||
but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | ||
Lesser General Public License for more details. | ||
You should have received a copy of the GNU Lesser General Public | ||
License along with the GNU C Library; if not, see | ||
<https://www.gnu.org/licenses/>. */ | ||
|
||
#ifndef _SIGNAL_H | ||
# error "Never include <bits/sigstksz.h> directly; use <signal.h> instead." | ||
#endif |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
#include_next <bits/sigstack.h> | ||
|
||
#if !defined _ISOMAC && !defined CONSTANT_MINSIGSTKSZ | ||
# define CONSTANT_MINSIGSTKSZ MINSIGSTKSZ | ||
#endif |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
/* NB: Don't define MINSIGSTKSZ nor SIGSTKSZ to sysconf (SC_SIGSTKSZ) for | ||
glibc build. IS_IN can only be used when _ISOMAC isn't defined. */ | ||
#ifdef _ISOMAC | ||
# include_next <bits/sigstksz.h> | ||
#elif IS_IN (libsupport) | ||
# include_next <bits/sigstksz.h> | ||
#endif |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,84 @@ | ||
/* Test of signal delivery on an alternate stack with MINSIGSTKSZ size. | ||
Copyright (C) 2020 Free Software Foundation, Inc. | ||
This file is part of the GNU C Library. | ||
The GNU C Library is free software; you can redistribute it and/or | ||
modify it under the terms of the GNU Lesser General Public | ||
License as published by the Free Software Foundation; either | ||
version 2.1 of the License, or (at your option) any later version. | ||
The GNU C Library is distributed in the hope that it will be useful, | ||
but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | ||
Lesser General Public License for more details. | ||
You should have received a copy of the GNU Lesser General Public | ||
License along with the GNU C Library; if not, see | ||
<https://www.gnu.org/licenses/>. */ | ||
|
||
#include <signal.h> | ||
#include <stdlib.h> | ||
#include <string.h> | ||
#include <unistd.h> | ||
#include <support/check.h> | ||
#include <support/support.h> | ||
|
||
static volatile sig_atomic_t handler_run; | ||
|
||
static void | ||
handler (int signo) | ||
{ | ||
/* Clear a bit of on-stack memory. */ | ||
volatile char buffer[256]; | ||
for (size_t i = 0; i < sizeof (buffer); ++i) | ||
buffer[i] = 0; | ||
handler_run = 1; | ||
} | ||
|
||
int | ||
do_test (void) | ||
{ | ||
size_t stack_buffer_size = 64 * 1024 * 1024; | ||
void *stack_buffer = xmalloc (stack_buffer_size); | ||
void *stack_end = stack_buffer + stack_buffer_size; | ||
memset (stack_buffer, 0xCC, stack_buffer_size); | ||
|
||
void *stack_bottom = stack_buffer + (stack_buffer_size + MINSIGSTKSZ) / 2; | ||
void *stack_top = stack_bottom + MINSIGSTKSZ; | ||
stack_t stack = | ||
{ | ||
.ss_sp = stack_bottom, | ||
.ss_size = MINSIGSTKSZ, | ||
}; | ||
if (sigaltstack (&stack, NULL) < 0) | ||
FAIL_RET ("sigaltstack: %m\n"); | ||
|
||
struct sigaction act = | ||
{ | ||
.sa_handler = handler, | ||
.sa_flags = SA_ONSTACK, | ||
}; | ||
if (sigaction (SIGUSR1, &act, NULL) < 0) | ||
FAIL_RET ("sigaction: %m\n"); | ||
|
||
if (kill (getpid (), SIGUSR1) < 0) | ||
FAIL_RET ("kill: %m\n"); | ||
|
||
if (handler_run != 1) | ||
FAIL_RET ("handler did not run\n"); | ||
|
||
for (void *p = stack_buffer; p < stack_bottom; ++p) | ||
if (*(unsigned char *) p != 0xCC) | ||
FAIL_RET ("changed byte %ld bytes below configured stack\n", | ||
(long) (stack_bottom - p)); | ||
for (void *p = stack_top; p < stack_end; ++p) | ||
if (*(unsigned char *) p != 0xCC) | ||
FAIL_RET ("changed byte %ld bytes above configured stack\n", | ||
(long) (p - stack_top)); | ||
|
||
free (stack_buffer); | ||
|
||
return 0; | ||
} | ||
|
||
#include <support/test-driver.c> |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
/* Definition of MINSIGSTKSZ and SIGSTKSZ. Linux version. | ||
Copyright (C) 2020 Free Software Foundation, Inc. | ||
This file is part of the GNU C Library. | ||
The GNU C Library is free software; you can redistribute it and/or | ||
modify it under the terms of the GNU Lesser General Public | ||
License as published by the Free Software Foundation; either | ||
version 2.1 of the License, or (at your option) any later version. | ||
The GNU C Library is distributed in the hope that it will be useful, | ||
but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | ||
Lesser General Public License for more details. | ||
You should have received a copy of the GNU Lesser General Public | ||
License along with the GNU C Library; if not, see | ||
<https://www.gnu.org/licenses/>. */ | ||
|
||
#ifndef _SIGNAL_H | ||
# error "Never include <bits/sigstksz.h> directly; use <signal.h> instead." | ||
#endif | ||
|
||
#if defined __USE_SC_SIGSTKSZ && __USE_SC_SIGSTKSZ | ||
# include <unistd.h> | ||
|
||
/* Default stack size for a signal handler: sysconf (SC_SIGSTKSZ). */ | ||
# undef SIGSTKSZ | ||
# define SIGSTKSZ sysconf (_SC_SIGSTKSZ) | ||
|
||
/* Minimum stack size for a signal handler: SIGSTKSZ. */ | ||
# undef MINSIGSTKSZ | ||
# define MINSIGSTKSZ SIGSTKSZ | ||
#endif |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
/* sysconf_sigstksz (). Linux/ia64 version. | ||
Copyright (C) 2020 Free Software Foundation, Inc. | ||
This file is part of the GNU C Library. | ||
The GNU C Library is free software; you can redistribute it and/or | ||
modify it under the terms of the GNU Lesser General Public | ||
License as published by the Free Software Foundation; either | ||
version 2.1 of the License, or (at your option) any later version. | ||
The GNU C Library is distributed in the hope that it will be useful, | ||
but WITHOUT ANY WARRANTY; without even the implied warranty of | ||
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU | ||
Lesser General Public License for more details. | ||
You should have received a copy of the GNU Lesser General Public | ||
License along with the GNU C Library; if not, see | ||
<https://www.gnu.org/licenses/>. */ | ||
|
||
/* Return sysconf (_SC_SIGSTKSZ). */ | ||
|
||
static long int | ||
sysconf_sigstksz (void) | ||
{ | ||
_Static_assert (__builtin_constant_p (SIGSTKSZ), | ||
"SIGSTKSZ is constant"); | ||
return SIGSTKSZ; | ||
} |
Oops, something went wrong.