Make cwd required

Signed-off-by: Mrunal Patel <[email protected]>

Author: mrunalp

libcontainer/init_linux.go

@@ -134,10 +134,8 @@ func finalizeNamespace(config *initConfig) error {
 	if err := w.drop(); err != nil {
 		return err
 	}
-	if config.Cwd != "" {
-		if err := syscall.Chdir(config.Cwd); err != nil {
-			return err
-		}
+	if err := syscall.Chdir(config.Cwd); err != nil {
+		return err
 	}
 	return nil
 }

spec.go

@@ -53,6 +53,7 @@ var specCommand = cli.Command{
 						"PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin",
 						"TERM=xterm",
 					},
+					Cwd: "/",
 				},
 				Hostname: "shell",
 				Mounts: []specs.MountPoint{

start.go

@@ -108,6 +108,10 @@ func startContainer(context *cli.Context, spec *specs.LinuxSpec, rspec *specs.Li
 	// ensure that the container is always removed if we were the process
 	// that created it.
 	defer destroy(container)
+
+	if spec.Process.Cwd == "" {
+		return -1, fmt.Errorf("Cwd property must not be empty")
+	}
 	process := newProcess(spec.Process)
 
 	// Support on-demand socket activation by passing file descriptors into the container init process.