From 33f1741ccbc084ded689693774db379d52a5e32d Mon Sep 17 00:00:00 2001 From: Aleksandr Nogikh Date: Thu, 10 Feb 2022 18:46:54 +0000 Subject: tools/syz-testbed: do Start()/Wait() instead of Run() Otherwise there's a risk that the instance is stopped before Run() has had a chance to be executed. This leads to a panic while doing cmd.Signal(). --- tools/syz-testbed/instance.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'tools') diff --git a/tools/syz-testbed/instance.go b/tools/syz-testbed/instance.go index bd25d55eb..888ad9ff7 100644 --- a/tools/syz-testbed/instance.go +++ b/tools/syz-testbed/instance.go @@ -38,8 +38,9 @@ func (inst *Instance) Run() error { cmd.Stderr = logfile complete := make(chan error) + cmd.Start() go func() { - complete <- cmd.Run() + complete <- cmd.Wait() }() select { -- cgit mrf-deployment