diff options
author | Chaoqun Han <hanssccv@gmail.com> | 2021-11-23 22:05:40 +0800 |
---|---|---|
committer | Ian Lance Taylor <iant@golang.org> | 2021-11-23 21:39:32 +0000 |
commit | e3eaedb5cf623d0836533573db4140749da42768 (patch) | |
tree | da0c3f28541b4060b475a23cd8f37294e925ddb6 /src/os | |
parent | 0f64c21d90c7017df4f199a5852d60d4b474c03c (diff) | |
download | go-e3eaedb5cf623d0836533573db4140749da42768.tar.gz go-e3eaedb5cf623d0836533573db4140749da42768.zip |
os/signal: reset SIGURG in TestSignal
Accepting SIGURG signals could cause SIGURG to take up the entire
channel buffer.
Enhance the stability of test cases by:
1. Stop accepting the SIGURG signal by adding ‘Reset(sys call.SIGURG)’
2. Close the c1 chan by adding ‘defer Stop(c1)’ (Another bug, NOT this bug)
Fixes #49724
Change-Id: I909a9993f0f6dd109c15e48a861683b87dfc4ab3
Reviewed-on: https://go-review.googlesource.com/c/go/+/366514
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Bryan C. Mills <bcmills@google.com>
Trust: Bryan C. Mills <bcmills@google.com>
Diffstat (limited to 'src/os')
-rw-r--r-- | src/os/signal/signal_test.go | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/os/signal/signal_test.go b/src/os/signal/signal_test.go index 3e85d936f8..e6fb24c6a8 100644 --- a/src/os/signal/signal_test.go +++ b/src/os/signal/signal_test.go @@ -136,6 +136,9 @@ func TestSignal(t *testing.T) { // Using 10 is arbitrary. c1 := make(chan os.Signal, 10) Notify(c1) + // Stop relaying the SIGURG signals. See #49724 + Reset(syscall.SIGURG) + defer Stop(c1) // Send this process a SIGWINCH t.Logf("sigwinch...") |