Przeglądaj źródła

Reorder code slightly in setupWorkingDirectory

Signed-off-by: Alexander Morozov <lk4d4@docker.com>
Alexander Morozov 9 lat temu
rodzic
commit
bb66c16871
1 zmienionych plików z 17 dodań i 16 usunięć
  1. 17 16
      daemon/container_unix.go

+ 17 - 16
daemon/container_unix.go

@@ -1105,28 +1105,29 @@ func (container *Container) getIpcContainer() (*Container, error) {
 }
 }
 
 
 func (container *Container) setupWorkingDirectory() error {
 func (container *Container) setupWorkingDirectory() error {
-	if container.Config.WorkingDir != "" {
-		container.Config.WorkingDir = filepath.Clean(container.Config.WorkingDir)
+	if container.Config.WorkingDir == "" {
+		return nil
+	}
+	container.Config.WorkingDir = filepath.Clean(container.Config.WorkingDir)
 
 
-		pth, err := container.GetResourcePath(container.Config.WorkingDir)
-		if err != nil {
+	pth, err := container.GetResourcePath(container.Config.WorkingDir)
+	if err != nil {
+		return err
+	}
+
+	pthInfo, err := os.Stat(pth)
+	if err != nil {
+		if !os.IsNotExist(err) {
 			return err
 			return err
 		}
 		}
 
 
-		pthInfo, err := os.Stat(pth)
-		if err != nil {
-			if !os.IsNotExist(err) {
-				return err
-			}
-
-			if err := system.MkdirAll(pth, 0755); err != nil {
-				return err
-			}
-		}
-		if pthInfo != nil && !pthInfo.IsDir() {
-			return derr.ErrorCodeNotADir.WithArgs(container.Config.WorkingDir)
+		if err := system.MkdirAll(pth, 0755); err != nil {
+			return err
 		}
 		}
 	}
 	}
+	if pthInfo != nil && !pthInfo.IsDir() {
+		return derr.ErrorCodeNotADir.WithArgs(container.Config.WorkingDir)
+	}
 	return nil
 	return nil
 }
 }