summaryrefslogtreecommitdiff
path: root/src/fsck
diff options
context:
space:
mode:
authorLennart Poettering <lennart@poettering.net>2015-03-09 19:57:39 +0100
committerLennart Poettering <lennart@poettering.net>2015-03-09 19:57:39 +0100
commit566690fdb0020c6038c2130ac108e18d4cd885f4 (patch)
treeb86f8b6cf5e5b70546d3b71ee7b618d41002e0cd /src/fsck
parent1952708abb10a47960d9160205e511343894896d (diff)
fsck: modernize error handling
Diffstat (limited to 'src/fsck')
-rw-r--r--src/fsck/fsck.c62
1 files changed, 36 insertions, 26 deletions
diff --git a/src/fsck/fsck.c b/src/fsck/fsck.c
index f3e90a8dbf..6e46633025 100644
--- a/src/fsck/fsck.c
+++ b/src/fsck/fsck.c
@@ -224,13 +224,15 @@ int main(int argc, char *argv[]) {
test_files();
- if (!arg_force && arg_skip)
- return 0;
+ if (!arg_force && arg_skip) {
+ r = 0;
+ goto finish;
+ }
udev = udev_new();
if (!udev) {
- log_oom();
- return EXIT_FAILURE;
+ r = log_oom();
+ goto finish;
}
if (argc > 1) {
@@ -238,14 +240,14 @@ int main(int argc, char *argv[]) {
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) {
- 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];
@@ -253,32 +255,37 @@ int main(int argc, char *argv[]) {
/* 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 */
- 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.");
- return EXIT_SUCCESS;
+ r = 0;
+ goto finish;
}
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.");
- return EXIT_FAILURE;
+ r = -ENXIO;
+ goto finish;
}
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);
- 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) {
- log_error_errno(errno, "pipe(): %m");
- return EXIT_FAILURE;
+ r = log_error_errno(errno, "pipe(): %m");
+ goto finish;
}
cmdline[i++] = "/sbin/fsck";
@@ -324,7 +332,7 @@ int main(int argc, char *argv[]) {
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 */
@@ -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;
- 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;
}
@@ -354,6 +362,8 @@ int main(int argc, char *argv[]) {
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);
@@ -361,17 +371,17 @@ int main(int argc, char *argv[]) {
/* Some other problem */
start_target(SPECIAL_EMERGENCY_TARGET);
else {
- r = EXIT_SUCCESS;
+ r = 0;
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:
- return r;
+ return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS;
}