Skip to content

Commit fa57c34

Browse files
authored
Merge pull request #108 from crackcomm/master
machine: stop signal listener
2 parents 8418667 + d07e42b commit fa57c34

File tree

1 file changed

+6
-3
lines changed

1 file changed

+6
-3
lines changed

machine.go

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -385,9 +385,10 @@ func (m *Machine) startVMM(ctx context.Context) error {
385385
syscall.SIGABRT)
386386
m.logger.Debugf("Setting up signal handler")
387387
go func() {
388-
sig := <-sigchan
389-
m.logger.Printf("Caught signal %s", sig)
390-
m.cmd.Process.Signal(sig)
388+
if sig, ok := <-sigchan; ok {
389+
m.logger.Printf("Caught signal %s", sig)
390+
m.cmd.Process.Signal(sig)
391+
}
391392
}()
392393

393394
// Wait for firecracker to initialize:
@@ -406,6 +407,8 @@ func (m *Machine) startVMM(ctx context.Context) error {
406407
m.err = err
407408
}
408409

410+
signal.Stop(sigchan)
411+
close(sigchan)
409412
close(m.exitCh)
410413
}()
411414

0 commit comments

Comments
 (0)