diff options
author | Ian Lance Taylor <iant@golang.org> | 2021-06-17 17:34:18 -0700 |
---|---|---|
committer | Ian Lance Taylor <iant@golang.org> | 2021-06-18 16:56:48 +0000 |
commit | 57aaa19aae32a6c75a07ea8e3006fdcda7583385 (patch) | |
tree | 7b49ee7f11d1213d2c6453ebfd9a04f0b6197211 /src/runtime/signal_unix.go | |
parent | 6f22d2c682d5fb7e8c4e9d2ab3e6a762756c1e30 (diff) | |
download | go-57aaa19aae32a6c75a07ea8e3006fdcda7583385.tar.gz go-57aaa19aae32a6c75a07ea8e3006fdcda7583385.zip |
runtime: disable CPU profiling before removing the SIGPROF handler
Otherwise, in c-archive or c-shared mode, there is the chance of
getting a SIGPROF just after the signal handler is removed but before
profiling is disabled, in which case the program will die.
Fixes #46498
Change-Id: I5492beef45fec9fb9a7f58724356d6aedaf799ac
Reviewed-on: https://go-review.googlesource.com/c/go/+/329290
Trust: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
Diffstat (limited to 'src/runtime/signal_unix.go')
-rw-r--r-- | src/runtime/signal_unix.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/runtime/signal_unix.go b/src/runtime/signal_unix.go index f2e526973d..03e2a40ce6 100644 --- a/src/runtime/signal_unix.go +++ b/src/runtime/signal_unix.go @@ -280,6 +280,8 @@ func setProcessCPUProfiler(hz int32) { it.it_value = it.it_interval setitimer(_ITIMER_PROF, &it, nil) } else { + setitimer(_ITIMER_PROF, &itimerval{}, nil) + // If the Go signal handler should be disabled by default, // switch back to the signal handler that was installed // when we enabled profiling. We don't try to handle the case @@ -303,8 +305,6 @@ func setProcessCPUProfiler(hz int32) { setsig(_SIGPROF, h) } } - - setitimer(_ITIMER_PROF, &itimerval{}, nil) } } |