Merge pull request #1549 from saschagrunert/unused-result

Fix warning around unused result on chdir("/")
containers · Sep 5, 2024 · df78c16 · df78c16 · packit-as-a-service · Sep 5, 2024
2 parents 841a0de + 23d5e49
commit df78c16
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/libcrun/linux.c b/src/libcrun/linux.c
@@ -5773,7 +5773,10 @@ libcrun_safe_chdir (const char *path, libcrun_error_t *err)
   /* Enforce that the returned path is an absolute path.  */
   if (ret == 0 || buffer[0] != '/')
     {
-      (void) chdir ("/");
+      if (chdir ("/") < 0)
+        {
+          /* Braces around empty body, to fix warning for [-Wunused-result] and error for [-Werror=empty-body]. */
+        }
       errno = ENOENT;
 
       /*