mirror of
https://github.com/golang/go
synced 2024-11-23 20:20:01 -07:00
62e1302267
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 #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>
36 lines
929 B
Go
36 lines
929 B
Go
// Copyright 2017 The Go Authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style
|
|
// license that can be found in the LICENSE file.
|
|
|
|
package main
|
|
|
|
// This program hung when run under the C/C++ ThreadSanitizer. TSAN installs a
|
|
// libc interceptor that writes signal handlers to a global variable within the
|
|
// TSAN runtime instead of making a sigaction system call. A bug in
|
|
// syscall.runtime_AfterForkInChild corrupted TSAN's signal forwarding table
|
|
// during calls to (*os/exec.Cmd).Run, causing the parent process to fail to
|
|
// invoke signal handlers.
|
|
|
|
import (
|
|
"fmt"
|
|
"os"
|
|
"os/exec"
|
|
"os/signal"
|
|
"syscall"
|
|
)
|
|
|
|
import "C"
|
|
|
|
func main() {
|
|
ch := make(chan os.Signal, 1)
|
|
signal.Notify(ch, syscall.SIGUSR1)
|
|
|
|
if err := exec.Command("true").Run(); err != nil {
|
|
fmt.Fprintf(os.Stderr, "Unexpected error from `true`: %v", err)
|
|
os.Exit(1)
|
|
}
|
|
|
|
syscall.Kill(syscall.Getpid(), syscall.SIGUSR1)
|
|
<-ch
|
|
}
|