diff options
author | Cherry Mui <cherryyz@google.com> | 2022-05-23 20:20:07 -0400 |
---|---|---|
committer | Alex Rakoczy <alex@golang.org> | 2022-05-25 19:25:04 +0000 |
commit | 65701ad2b430466dd4bd6e1df107f81c0f8ee9cb (patch) | |
tree | 917e799561b04243438bf6ae12b5a3a9f851aa2c | |
parent | e846f3f2d602ec5fd4689e14d7530d894b807e70 (diff) | |
download | go-65701ad2b430466dd4bd6e1df107f81c0f8ee9cb.tar.gz go-65701ad2b430466dd4bd6e1df107f81c0f8ee9cb.zip |
[release-branch.go1.17] misc/cgo/testsanitizers: use buffered channel in tsan12.go
os/signal.Notify requires that "the caller must ensure that c has
sufficient buffer space to keep up with the expected signal rate"
as it does a nonblocking send when it receives a signal. The test
currently using a unbuffered channel, which means it may miss the
signal if the signal arrives before the channel receive operation.
Fixes #53042.
Updates #52998.
Change-Id: Icdcab9396d735506480ef880fb45a4669fa7cc8f
Reviewed-on: https://go-review.googlesource.com/c/go/+/407888
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Bryan Mills <bcmills@google.com>
Run-TryBot: Cherry Mui <cherryyz@google.com>
Reviewed-by: Cuong Manh Le <cuong.manhle.vn@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
(cherry picked from commit 62e130226767a088ace196da90a774c9a9d14689)
Reviewed-on: https://go-review.googlesource.com/c/go/+/408115
-rw-r--r-- | misc/cgo/testsanitizers/testdata/tsan12.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/misc/cgo/testsanitizers/testdata/tsan12.go b/misc/cgo/testsanitizers/testdata/tsan12.go index 3e767eee1f..0ef545d09b 100644 --- a/misc/cgo/testsanitizers/testdata/tsan12.go +++ b/misc/cgo/testsanitizers/testdata/tsan12.go @@ -22,7 +22,7 @@ import ( import "C" func main() { - ch := make(chan os.Signal) + ch := make(chan os.Signal, 1) signal.Notify(ch, syscall.SIGUSR1) if err := exec.Command("true").Run(); err != nil { |