diff options
| author | June McEnroe <june@causal.agency> | 2020-08-15 23:11:09 -0400 |
|---|---|---|
| committer | June McEnroe <june@causal.agency> | 2020-08-15 23:11:09 -0400 |
| commit | 1c4dd05f407633b9d8ca156a7ee4984e1b131cf6 (patch) | |
| tree | 8a15e665d2d9686dc6185f8c489d8fb744173ddd /service.c | |
| parent | Remove EX_CONFIG from default stopexits (diff) | |
| download | catsit-1c4dd05f407633b9d8ca156a7ee4984e1b131cf6.tar.gz catsit-1c4dd05f407633b9d8ca156a7ee4984e1b131cf6.zip | |
Add 126 to hardcoded stop exits
> If a command is not found, the exit status shall be 127. If the > command name is found, but it is not an executable utility, the exit > status shall be 126. Applications that invoke utilities without using > the shell should use these exit status values to report similar errors. [1]: https://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html#tag_18_08_02
Diffstat (limited to 'service.c')
| -rw-r--r-- | service.c | 19 |
1 files changed, 14 insertions, 5 deletions
diff --git a/service.c b/service.c index e7cfe2f..63a647c 100644 --- a/service.c +++ b/service.c @@ -34,6 +34,11 @@ #include "daemon.h" +enum { + ExitNotFound = 127, + ExitNoExec = 126, +}; + const char *serviceDir = "/"; uid_t serviceUID; gid_t serviceGID; @@ -173,13 +178,13 @@ void serviceStart(struct Service *service) { dup2(service->errPipe[1], STDERR_FILENO); int error = chdir(serviceDir); - if (error) err(StopExit, "%s", serviceDir); + if (error) err(ExitNoExec, "%s", serviceDir); error = setgid(serviceGID); - if (error) err(StopExit, "setgid"); + if (error) err(ExitNoExec, "setgid"); error = setuid(serviceUID); - if (error) err(StopExit, "setuid"); + if (error) err(ExitNoExec, "setuid"); size_t len = 0; char command[ARG_MAX]; @@ -197,7 +202,7 @@ void serviceStart(struct Service *service) { _PATH_BSHELL, "-c", command, service->name, NULL, serviceEnviron ); - err(StopExit, "%s", _PATH_BSHELL); + err(ExitNotFound, "%s", _PATH_BSHELL); } void serviceSignal(struct Service *service, int signal) { @@ -266,7 +271,11 @@ void serviceReap(pid_t pid, int status) { service->state = Stop; if (WIFEXITED(status)) { int exit = WEXITSTATUS(status); - if (exit == StopExit || setTest(&stopExits, exit)) { + if ( + exit == ExitNotFound || + exit == ExitNoExec || + setTest(&stopExits, exit) + ) { service->intent = Stop; } if (exit) { |