procd: replace exit(-1) with exit(EXIT_FAILURE)
authorMichael Heimpold <mhei@heimpold.de>
Tue, 1 Jan 2019 23:44:59 +0000 (00:44 +0100)
committerJohn Crispin <john@phrozen.org>
Sun, 27 Oct 2019 13:22:27 +0000 (14:22 +0100)
Signed-off-by: Michael Heimpold <mhei@heimpold.de>
initd/init.c
initd/preinit.c
initd/zram.c
plug/coldplug.c
plug/hotplug.c
upgraded/upgraded.c

index b253fb6..9b47826 100644 (file)
@@ -92,7 +92,7 @@ main(int argc, char **argv)
 
                execvp(kmod[0], kmod);
                ERROR("Failed to start kmodloader: %m\n");
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
        if (pid <= 0) {
                ERROR("Failed to start kmodloader instance: %m\n");
index 84e62b0..9dfe5c1 100644 (file)
@@ -23,7 +23,7 @@
 #include <libubus.h>
 
 #include <stdio.h>
-
+#include <stdlib.h>
 #include <unistd.h>
 
 #include "init.h"
@@ -135,7 +135,7 @@ preinit(void)
        if (!plugd_proc.pid) {
                execvp(plug[0], plug);
                ERROR("Failed to start plugd: %m\n");
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
        if (plugd_proc.pid <= 0) {
                ERROR("Failed to start new plugd instance: %m\n");
@@ -157,7 +157,7 @@ preinit(void)
        if (!preinit_proc.pid) {
                execvp(init[0], init);
                ERROR("Failed to start preinit: %m\n");
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
        if (preinit_proc.pid <= 0) {
                ERROR("Failed to start new preinit instance: %m\n");
index 487d3d6..380fe0e 100644 (file)
@@ -66,7 +66,7 @@ early_insmod(char *module)
                modprobe[1] = path;
                execvp(modprobe[0], modprobe);
                ERROR("Can't exec %s: %m\n", modprobe[0]);
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
 
        if (pid <= 0) {
@@ -109,7 +109,7 @@ mount_zram_on_tmp(void)
        if (!pid) {
                execvp(mkfs[0], mkfs);
                ERROR("Can't exec %s: %m\n", mkfs[0]);
-               exit(-1);
+               exit(EXIT_FAILURE);
        } else if (pid <= 0) {
                ERROR("Can't exec %s: %m\n", mkfs[0]);
                return -1;
index 12df421..b185083 100644 (file)
@@ -16,6 +16,7 @@
 #include <sys/types.h>
 #include <sys/mount.h>
 
+#include <stdlib.h>
 #include <unistd.h>
 
 #include "../procd.h"
@@ -59,7 +60,7 @@ void procd_coldplug(void)
        if (!udevtrigger.pid) {
                execvp(argv[0], argv);
                ERROR("Failed to start coldplug: %m\n");
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
 
        if (udevtrigger.pid <= 0) {
index c572ced..de0511f 100644 (file)
@@ -221,7 +221,7 @@ static void handle_exec(struct blob_attr *msg, struct blob_attr *data)
                argv[i] = NULL;
                execvp(argv[0], &argv[0]);
        }
-       exit(-1);
+       exit(EXIT_FAILURE);
 }
 
 static void handle_button_start(struct blob_attr *msg, struct blob_attr *data)
@@ -246,7 +246,7 @@ static void handle_firmware(struct blob_attr *msg, struct blob_attr *data)
 
        if (!file || !dir || !dev) {
                ERROR("Request for unknown firmware %s/%s\n", dir, file);
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
 
        path = alloca(strlen(dir) + strlen(file) + 2);
@@ -271,11 +271,11 @@ send_to_kernel:
        load = open(loadpath, O_WRONLY);
        if (!load) {
                ERROR("Failed to open %s: %m\n", loadpath);
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
        if (write(load, "1", 1) == -1) {
                ERROR("Failed to write to %s: %m\n", loadpath);
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
        close(load);
 
@@ -283,7 +283,7 @@ send_to_kernel:
        fw = open(syspath, O_WRONLY);
        if (fw < 0) {
                ERROR("Failed to open %s: %m\n", syspath);
-               exit(-1);
+               exit(EXIT_FAILURE);
        }
 
        len = s.st_size;
@@ -309,7 +309,7 @@ send_to_kernel:
 
        DEBUG(2, "Done loading %s\n", path);
 
-       exit(-1);
+       exit(EXIT_FAILURE);
 }
 
 static void handle_start_console(struct blob_attr *msg, struct blob_attr *data)
@@ -323,7 +323,7 @@ static void handle_start_console(struct blob_attr *msg, struct blob_attr *data)
 
        DEBUG(2, "Done starting console for %s\n", dev);
 
-       exit(-1);
+       exit(EXIT_FAILURE);
 }
 
 enum {
index 0b82f20..db98701 100644 (file)
@@ -56,7 +56,7 @@ static void sysupgrade(char *path, char *command)
                /* Child */
                execvp(args[0], args);
                fprintf(stderr, "Failed to exec sysupgrade\n");
-               _exit(-1);
+               _exit(EXIT_FAILURE);
        }
 
        uloop_process_add(&upgrade_proc);