diff options
| author | Eelco Dolstra <edolstra@gmail.com> | 2021-11-04 10:44:33 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-11-04 10:44:33 +0100 |
| commit | c5fd0b46ae56447d9d0a2028d464a5bbf753bf55 (patch) | |
| tree | 6e87c29fa2005a5de4fc1ada651e5ee5cf003d3f /boehmgc-coroutine-sp-fallback.diff | |
| parent | 3f447bcd5f42f0503dcc0a5dd96737b11ec5dd14 (diff) | |
| parent | 6f291ed718868a926c6c1aacac12359a78c60da7 (diff) | |
Merge pull request #5486 from mohe2015/bugfixes2
Fix leaking pthread_attr_t
Diffstat (limited to 'boehmgc-coroutine-sp-fallback.diff')
| -rw-r--r-- | boehmgc-coroutine-sp-fallback.diff | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/boehmgc-coroutine-sp-fallback.diff b/boehmgc-coroutine-sp-fallback.diff index fa8dd0325..e659bf470 100644 --- a/boehmgc-coroutine-sp-fallback.diff +++ b/boehmgc-coroutine-sp-fallback.diff @@ -1,8 +1,8 @@ diff --git a/pthread_stop_world.c b/pthread_stop_world.c -index 1cee6a0b..46c3acd9 100644 +index 4b2c429..1fb4c52 100644 --- a/pthread_stop_world.c +++ b/pthread_stop_world.c -@@ -674,6 +674,8 @@ GC_INNER void GC_push_all_stacks(void) +@@ -673,6 +673,8 @@ GC_INNER void GC_push_all_stacks(void) struct GC_traced_stack_sect_s *traced_stack_sect; pthread_t self = pthread_self(); word total_size = 0; @@ -11,7 +11,7 @@ index 1cee6a0b..46c3acd9 100644 if (!EXPECT(GC_thr_initialized, TRUE)) GC_thr_init(); -@@ -723,6 +725,28 @@ GC_INNER void GC_push_all_stacks(void) +@@ -722,6 +724,31 @@ GC_INNER void GC_push_all_stacks(void) hi = p->altstack + p->altstack_size; /* FIXME: Need to scan the normal stack too, but how ? */ /* FIXME: Assume stack grows down */ @@ -22,6 +22,9 @@ index 1cee6a0b..46c3acd9 100644 + if (pthread_attr_getstacksize(&pattr, &stack_limit)) { + ABORT("GC_push_all_stacks: pthread_attr_getstacksize failed!"); + } ++ if (pthread_attr_destroy(&pattr)) { ++ ABORT("GC_push_all_stacks: pthread_attr_destroy failed!"); ++ } + // When a thread goes into a coroutine, we lose its original sp until + // control flow returns to the thread. + // While in the coroutine, the sp points outside the thread stack, |