diff --git a/libcontainer/configs/config.go b/libcontainer/configs/config.go index 14a096038..c6b259e9f 100644 --- a/libcontainer/configs/config.go +++ b/libcontainer/configs/config.go @@ -387,7 +387,7 @@ func (c Command) Run(s *specs.State) error { case err := <-errC: return err case <-timerCh: - cmd.Process.Kill() + _ = cmd.Process.Kill() <-errC return fmt.Errorf("hook ran past specified timeout of %.1fs", c.Timeout.Seconds()) } diff --git a/libcontainer/configs/config_test.go b/libcontainer/configs/config_test.go index 97286740f..a270f3780 100644 --- a/libcontainer/configs/config_test.go +++ b/libcontainer/configs/config_test.go @@ -145,12 +145,15 @@ func TestFuncHookRun(t *testing.T) { fHook := configs.NewFunctionHook(func(s *specs.State) error { if !reflect.DeepEqual(state, s) { - t.Errorf("Expected state %+v to equal %+v", state, s) + return fmt.Errorf("expected state %+v to equal %+v", state, s) } return nil }) - fHook.Run(state) + err := fHook.Run(state) + if err != nil { + t.Fatal(err) + } } func TestCommandHookRun(t *testing.T) {