mirrors/libguestfs
1
0
Fork 0
mirror of https://github.com/libguestfs/libguestfs.git synced 2026-03-21 22:53:37 +00:00
Code Issues Packages Projects Releases Wiki Activity

lib: command: If command fails, exit with 126 or 127 as defined by POSIX.

Browse Source 
If running the external command fails in "argv mode" (ie. when
not using the shell), then exit with either 126 or 127 as defined
by POSIX.

This is mostly the same as what bash does, see
execute_cmd.c:shell_execve in the bash sources.

Note: saving errno around perror(3) if necessary, otherwise you will
see different behaviour between verbose and non-verbose mode.  In
non-verbose mode, perror(3) tried to print to a closed file
descriptor, failing and overwriting errno.  That took some time to
debug.
This commit is contained in:
Richard W.M. Jones
2017-10-06 14:20:22 +01:00
parent 3ef6794b09
commit f2fe422909
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lib/command.c
View File

@@ -539,7 +539,7 @@ static void
run_child (struct command *cmd)
{
struct sigaction sa;
int i, fd, max_fd, r;
int i, err, fd, max_fd, r;
char status_string[80];
#ifdef HAVE_SETRLIMIT
struct child_rlimits *child_rlimit;
@@ -614,8 +614,12 @@ run_child (struct command *cmd)
switch (cmd->style) {
case COMMAND_STYLE_EXECV:
execvp (cmd->argv.argv[0], cmd->argv.argv);
err = errno;
perror (cmd->argv.argv[0]);
_exit (EXIT_FAILURE);
/* These error codes are defined in POSIX and meant to be the
* same as the shell.
*/
_exit (err == ENOENT ? 127 : 126);
case COMMAND_STYLE_SYSTEM:
r = system (cmd->string.str);