fix runc's poststart behaviour doesn't match the runtime-spec

Signed-off-by: ningmingxiao <[email protected]>

libcontainer/container_linux.go

@@ -225,8 +225,22 @@ func (c *Container) Run(process *Process) error {
 // Exec signals the container to exec the users process at the end of the init.
 func (c *Container) Exec() error {
  c.m.Lock()
- defer c.m.Unlock()
- return c.exec()
+ err := c.exec()
+ if err != nil {
+ c.m.Unlock()
+ return err
+ }
+ c.m.Unlock()
+ s, err := c.OCIState()
+ if err != nil {
+ return err
+ }
+ if c.Config().Hooks != nil {
+ if err := c.Config().Hooks.Run(configs.Poststart, s); err != nil {
+ return fmt.Errorf("run poststart hook %w", err)
+ }
+ }
+ return nil
 }
 
 func (c *Container) exec() error {
@@ -353,19 +367,6 @@ func (c *Container) start(process *Process) (retErr error) {
 
  if process.Init {
  c.fifo.Close()
- if c.config.Hooks != nil {
- s, err := c.currentOCIState()
- if err != nil {
- return err
- }
-
- if err := c.config.Hooks.Run(configs.Poststart, s); err != nil {
- if err := ignoreTerminateErrors(parent.terminate()); err != nil {
- logrus.Warn(fmt.Errorf("error running poststart hook: %w", err))
- }
- return err
- }
- }
  }
  return nil
 }