fsck: modernize error handling

[elogind.git] / src / fsck / fsck.c

diff --git a/src/fsck/fsck.c b/src/fsck/fsck.c
index f3e90a8dbf3781de34d3bda487e57d4f220ad884..6e46633025c3c152f78ed89f454a36b50fb4029f 100644 (file)
--- a/src/fsck/fsck.c
+++ b/src/fsck/fsck.c
@@ -224,13 +224,15 @@ int main(int argc, char *argv[]) {
 
         test_files();
 
 
         test_files();
 

-        if (!arg_force && arg_skip)
-                return 0;
+        if (!arg_force && arg_skip) {
+                r = 0;
+                goto finish;
+        }

 
         udev = udev_new();
         if (!udev) {
 
         udev = udev_new();
         if (!udev) {

-                log_oom();
-                return EXIT_FAILURE;
+                r = log_oom();
+                goto finish;

         }
 
         if (argc > 1) {
         }
 
         if (argc > 1) {


@@ -238,14 +240,14 @@ int main(int argc, char *argv[]) {
                 root_directory = false;
 
                 if (stat(device, &st) < 0) {
                 root_directory = false;
 
                 if (stat(device, &st) < 0) {

-                        log_error_errno(errno, "Failed to stat '%s': %m", device);
-                        return EXIT_FAILURE;
+                        r = log_error_errno(errno, "Failed to stat '%s': %m", device);
+                        goto finish;

                 }
 
                 udev_device = udev_device_new_from_devnum(udev, 'b', st.st_rdev);
                 if (!udev_device) {
                 }
 
                 udev_device = udev_device_new_from_devnum(udev, 'b', st.st_rdev);
                 if (!udev_device) {

-                        log_error("Failed to detect device %s", device);
-                        return EXIT_FAILURE;
+                        r = log_error_errno(errno, "Failed to detect device %s", device);
+                        goto finish;

                 }
         } else {
                 struct timespec times[2];
                 }
         } else {
                 struct timespec times[2];


@@ -253,32 +255,37 @@ int main(int argc, char *argv[]) {
                 /* Find root device */
 
                 if (stat("/", &st) < 0) {
                 /* Find root device */
 
                 if (stat("/", &st) < 0) {

-                        log_error_errno(errno, "Failed to stat() the root directory: %m");
-                        return EXIT_FAILURE;
+                        r = log_error_errno(errno, "Failed to stat() the root directory: %m");
+                        goto finish;

                 }
 
                 /* Virtual root devices don't need an fsck */
                 }
 
                 /* Virtual root devices don't need an fsck */

-                if (major(st.st_dev) == 0)
-                        return EXIT_SUCCESS;
+                if (major(st.st_dev) == 0) {
+                        log_debug("Root directory is virtual, skipping check.");
+                        r = 0;
+                        goto finish;
+                }

 
                 /* check if we are already writable */
                 times[0] = st.st_atim;
                 times[1] = st.st_mtim;
                 if (utimensat(AT_FDCWD, "/", times, 0) == 0) {
                         log_info("Root directory is writable, skipping check.");
 
                 /* check if we are already writable */
                 times[0] = st.st_atim;
                 times[1] = st.st_mtim;
                 if (utimensat(AT_FDCWD, "/", times, 0) == 0) {
                         log_info("Root directory is writable, skipping check.");

-                        return EXIT_SUCCESS;
+                        r = 0;
+                        goto finish;

                 }
 
                 udev_device = udev_device_new_from_devnum(udev, 'b', st.st_dev);
                 if (!udev_device) {
                 }
 
                 udev_device = udev_device_new_from_devnum(udev, 'b', st.st_dev);
                 if (!udev_device) {

-                        log_error("Failed to detect root device.");
-                        return EXIT_FAILURE;
+                        r = log_error_errno(errno, "Failed to detect root device.");
+                        goto finish;

                 }
 
                 device = udev_device_get_devnode(udev_device);
                 if (!device) {
                         log_error("Failed to detect device node of root directory.");
                 }
 
                 device = udev_device_get_devnode(udev_device);
                 if (!device) {
                         log_error("Failed to detect device node of root directory.");

-                        return EXIT_FAILURE;
+                        r = -ENXIO;
+                        goto finish;

                 }
 
                 root_directory = true;
                 }
 
                 root_directory = true;


@@ -289,14 +296,15 @@ int main(int argc, char *argv[]) {
                 r = fsck_exists(type);
                 if (r == -ENOENT) {
                         log_info("fsck.%s doesn't exist, not checking file system on %s", type, device);
                 r = fsck_exists(type);
                 if (r == -ENOENT) {
                         log_info("fsck.%s doesn't exist, not checking file system on %s", type, device);

-                        return EXIT_SUCCESS;
+                        r = 0;
+                        goto finish;

                 } else if (r < 0)
                         log_warning_errno(r, "fsck.%s cannot be used for %s: %m", type, device);
         }
 
         if (pipe(progress_pipe) < 0) {
                 } else if (r < 0)
                         log_warning_errno(r, "fsck.%s cannot be used for %s: %m", type, device);
         }
 
         if (pipe(progress_pipe) < 0) {

-                log_error_errno(errno, "pipe(): %m");
-                return EXIT_FAILURE;
+                r = log_error_errno(errno, "pipe(): %m");
+                goto finish;

         }
 
         cmdline[i++] = "/sbin/fsck";
         }
 
         cmdline[i++] = "/sbin/fsck";


@@ -324,7 +332,7 @@ int main(int argc, char *argv[]) {
 
         pid = fork();
         if (pid < 0) {
 
         pid = fork();
         if (pid < 0) {

-                log_error_errno(errno, "fork(): %m");
+                r = log_error_errno(errno, "fork(): %m");

                 goto finish;
         } else if (pid == 0) {
                 /* Child */
                 goto finish;
         } else if (pid == 0) {
                 /* Child */


@@ -338,9 +346,9 @@ int main(int argc, char *argv[]) {
         progress_rc = process_progress(progress_pipe[0], pid, st.st_rdev);
         progress_pipe[0] = -1;
 
         progress_rc = process_progress(progress_pipe[0], pid, st.st_rdev);
         progress_pipe[0] = -1;
 

-        q = wait_for_terminate(pid, &status);
-        if (q < 0) {
-                log_error_errno(q, "waitid(): %m");
+        r = wait_for_terminate(pid, &status);
+        if (r < 0) {
+                log_error_errno(r, "waitid(): %m");

                 goto finish;
         }
 
                 goto finish;
         }
 


@@ -354,6 +362,8 @@ int main(int argc, char *argv[]) {
                 else if (progress_rc != 0)
                         log_error("fsck failed due to unknown reason.");
 
                 else if (progress_rc != 0)
                         log_error("fsck failed due to unknown reason.");
 

+                r = -EINVAL;
+

                 if (status.si_code == CLD_EXITED && (status.si_status & 2) && root_directory)
                         /* System should be rebooted. */
                         start_target(SPECIAL_REBOOT_TARGET);
                 if (status.si_code == CLD_EXITED && (status.si_status & 2) && root_directory)
                         /* System should be rebooted. */
                         start_target(SPECIAL_REBOOT_TARGET);


@@ -361,17 +371,17 @@ int main(int argc, char *argv[]) {
                         /* Some other problem */
                         start_target(SPECIAL_EMERGENCY_TARGET);
                 else {
                         /* Some other problem */
                         start_target(SPECIAL_EMERGENCY_TARGET);
                 else {

-                        r = EXIT_SUCCESS;
+                        r = 0;

                         if (progress_rc != 0)
                                 log_warning("Ignoring error.");
                 }
 
         } else
                         if (progress_rc != 0)
                                 log_warning("Ignoring error.");
                 }
 
         } else

-                r = EXIT_SUCCESS;
+                r = 0;

 
         if (status.si_code == CLD_EXITED && (status.si_status & 1))
                 touch("/run/systemd/quotacheck");
 
 finish:
 
         if (status.si_code == CLD_EXITED && (status.si_status & 1))
                 touch("/run/systemd/quotacheck");
 
 finish:

-        return r;
+        return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS;

 }
 }