summaryrefslogtreecommitdiff
path: root/signal.c
diff options
context:
space:
mode:
authornagachika <nagachika@ruby-lang.org>2021-09-11 13:59:43 +0900
committernagachika <nagachika@ruby-lang.org>2021-09-11 13:59:43 +0900
commit13f64b65e0476c2fe416a29274fcc91e3c0cf5d3 (patch)
treed867b4b485adb9ea441ea4b9e681719f93d24b24 /signal.c
parent8e4ed4ed00e8346029953c14dd414a996e37a437 (diff)
merge revision(s) f336a3eb6c76890f3d8f878725b3d328c8fdcf33: [Backport #18126]
Use free instead of xfree to free altstack The altstack memory of a thread may be free'ed even after the VM is destructed. After that, GC is no longer available, so calling xfree may lead to a segfault. This changeset uses the bare free function to free the altstack memory instead of xfree. [Bug #18126] --- signal.c | 5 ++++- vm_core.h | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-)
Diffstat (limited to 'signal.c')
-rw-r--r--signal.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/signal.c b/signal.c
index 6eea6aabe2..c13bc19f1c 100644
--- a/signal.c
+++ b/signal.c
@@ -557,10 +557,13 @@ static int rb_sigaltstack_size_value = 0;
void *
rb_allocate_sigaltstack(void)
{
+ void *altstack;
if (!rb_sigaltstack_size_value) {
rb_sigaltstack_size_value = rb_sigaltstack_size();
}
- return xmalloc(rb_sigaltstack_size_value);
+ altstack = malloc(rb_sigaltstack_size_value);
+ if (!altstack) rb_memerror();
+ return altstack;
}
/* alternate stack for SIGSEGV */