[PATCH 1/7] passt: Explicitly check return value of chdir()

[Stefano Brivio <sbrivio@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 773 bytes --]

...it doesn't actually matter as we're checking errno at the very
end, but, depending on build flags, chdir() might be declared with
warn_unused_result and the compiler issues a warning.

Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com>
---
 passt.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/passt.c b/passt.c
index 3d18d1f..038d50a 100644
--- a/passt.c
+++ b/passt.c
@@ -260,7 +260,9 @@ static int sandbox(struct ctx *c)
 	mount("", "/", "", MS_UNBINDABLE | MS_REC, NULL);
 	mount("", TMPDIR, "tmpfs", MS_NODEV | MS_NOEXEC | MS_NOSUID | MS_RDONLY,
 	      "nr_inodes=2,nr_blocks=0");
-	chdir(TMPDIR);
+	if (chdir(TMPDIR))
+		return -errno;
+
 	syscall(SYS_pivot_root, ".", ".");
 	umount2(".", MNT_DETACH | UMOUNT_NOFOLLOW);
 
-- 
@@ -260,7 +260,9 @@ static int sandbox(struct ctx *c)
 	mount("", "/", "", MS_UNBINDABLE | MS_REC, NULL);
 	mount("", TMPDIR, "tmpfs", MS_NODEV | MS_NOEXEC | MS_NOSUID | MS_RDONLY,
 	      "nr_inodes=2,nr_blocks=0");
-	chdir(TMPDIR);
+	if (chdir(TMPDIR))
+		return -errno;
+
 	syscall(SYS_pivot_root, ".", ".");
 	umount2(".", MNT_DETACH | UMOUNT_NOFOLLOW);
 
-- 
2.34.1