aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristian Brauner <brauner@kernel.org>2022-01-07 15:58:16 +0100
committerEryu Guan <guaneryu@gmail.com>2022-01-16 12:19:54 +0800
commitb3c31bd3ab60b630feb779fee454545dc47c7306 (patch)
tree039a88e154698228e46b43d48f05b316992f506e
parentc61c1e2f4cad89d4f900eaef173616b309228110 (diff)
downloadxfstests-dev-b3c31bd3ab60b630feb779fee454545dc47c7306.tar.gz
idmapped-mounts: remove redundant fchownat() call in setgid tests
There's another call to fchownat() right above it so we really don't need the second one. Cc: Seth Forshee <sforshee@digitalocean.com> Cc: Eryu Guan <guaneryu@gmail.com> Cc: Christoph Hellwig <hch@lst.de> Cc: fstests@vger.kernel.org Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Eryu Guan <guaneryu@gmail.com>
-rw-r--r--src/idmapped-mounts/idmapped-mounts.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/src/idmapped-mounts/idmapped-mounts.c b/src/idmapped-mounts/idmapped-mounts.c
index da69077904..56b26b0ce2 100644
--- a/src/idmapped-mounts/idmapped-mounts.c
+++ b/src/idmapped-mounts/idmapped-mounts.c
@@ -8133,11 +8133,6 @@ static int setgid_create_idmapped_in_userns(void)
goto out;
}
- if (fchownat(t_dir1_fd, "", -1, 1000, AT_SYMLINK_NOFOLLOW | AT_EMPTY_PATH)) {
- log_stderr("failure: fchownat");
- goto out;
- }
-
pid = fork();
if (pid < 0) {
log_stderr("failure: fork");