Merge pull request #567 from cgwalters/chdir-after-chroot

user-group: Add chdir("/") after chroot()
coreos · Jun 28, 2018 · f41d15d · f41d15d
2 parents 3c7dbd3 + 85e6b62
commit f41d15d
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/internal/exec/util/user_group_lookup.c b/internal/exec/util/user_group_lookup.c
@@ -58,6 +58,10 @@ static int user_lookup_fn(lookup_ctxt_t *ctxt) {
 		goto out_err;
 	}
 
+	if (chdir("/") == -1) {
+		goto out_err;
+	}
+
 	if(getpwnam_r(ctxt->name, &p, buf, sizeof(buf), &pptr) != 0 || !pptr) {
 		goto out_err;
 	}
@@ -90,6 +94,10 @@ static int group_lookup_fn(lookup_ctxt_t *ctxt) {
 		goto out_err;
 	}
 
+	if (chdir("/") == -1) {
+		goto out_err;
+	}
+
 	if(getgrnam_r(ctxt->name, &g, buf, sizeof(buf), &gptr) != 0 || !gptr) {
 		goto out_err;
 	}