Merge pull request #9399 from vrothberg/home-sweet-home

do not set empty $HOME
containers · Feb 16, 2021 · 7fb347a · 7fb347a
2 parents 58a4793 + 2ec0e3b
commit 7fb347a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/libpod/container_internal_linux.go b/libpod/container_internal_linux.go
@@ -466,7 +466,7 @@ func (c *Container) generateSpec(ctx context.Context) (*spec.Spec, error) {
 			break
 		}
 	}
-	if !hasHomeSet {
+	if !hasHomeSet && execUser.Home != "" {
 		c.config.Spec.Process.Env = append(c.config.Spec.Process.Env, fmt.Sprintf("HOME=%s", execUser.Home))
 	}
 

diff --git a/test/system/030-run.bats b/test/system/030-run.bats
@@ -662,4 +662,10 @@ json-file | f
     run_podman rm $cname
 }
 
+@test "podman run - do not set empty HOME" {
+    # Regression test for #9378.
+    run_podman run --rm --user 100 $IMAGE printenv
+    is "$output" ".*HOME=/.*"
+}
+
 # vim: filetype=sh
-Original file line number
+Diff line change
@@ Expand Up @@
     			break
     		}
     	}
-    	if !hasHomeSet {
+    	if !hasHomeSet && execUser.Home != "" {
     		c.config.Spec.Process.Env = append(c.config.Spec.Process.Env, fmt.Sprintf("HOME=%s", execUser.Home))
     	}
@@ Expand Down @@