diff options
author | Chet Ramey <chet.ramey@case.edu> | 2011-12-07 09:37:19 -0500 |
---|---|---|
committer | Chet Ramey <chet.ramey@case.edu> | 2011-12-07 09:37:19 -0500 |
commit | a05a1337fd1ddd41aa08f1b61f8351cda760b5b8 (patch) | |
tree | b639580457c54815b86178dbe5332f5a5155e397 /execute_cmd.c~ | |
parent | db31fb26445a2ea2e908bac08b247e41388d76a8 (diff) | |
download | bash-a05a1337fd1ddd41aa08f1b61f8351cda760b5b8.tar.gz |
commit bash-20090212 snapshot
Diffstat (limited to 'execute_cmd.c~')
-rw-r--r-- | execute_cmd.c~ | 52 |
1 files changed, 46 insertions, 6 deletions
diff --git a/execute_cmd.c~ b/execute_cmd.c~ index ac8a7a36..e1053846 100644 --- a/execute_cmd.c~ +++ b/execute_cmd.c~ @@ -513,7 +513,7 @@ execute_command_internal (command, asynchronous, pipe_in, pipe_out, int pipe_in, pipe_out; struct fd_bitmap *fds_to_close; { - int exec_result, invert, ignore_return, was_error_trap; + int exec_result, user_subshell, invert, ignore_return, was_error_trap; REDIRECT *my_undo_list, *exec_undo_list; volatile int last_pid; volatile int save_line_number; @@ -557,6 +557,8 @@ execute_command_internal (command, asynchronous, pipe_in, pipe_out, return (execute_coproc (command, pipe_in, pipe_out, fds_to_close)); #endif + user_subshell = command->type == cm_subshell || ((command->flags & CMD_WANT_SUBSHELL) != 0); + if (command->type == cm_subshell || (command->flags & (CMD_WANT_SUBSHELL|CMD_FORCE_SUBSHELL)) || (shell_control_structure (command->type) && @@ -590,6 +592,10 @@ execute_command_internal (command, asynchronous, pipe_in, pipe_out, if (asynchronous == 0) { + was_error_trap = signal_is_trapped (ERROR_TRAP) && signal_is_ignored (ERROR_TRAP) == 0; + invert = (command->flags & CMD_INVERT_RETURN) != 0; + ignore_return = (command->flags & CMD_IGNORE_RETURN) != 0; + last_command_exit_value = wait_for (paren_pid); /* If we have to, invert the return value. */ @@ -600,6 +606,21 @@ execute_command_internal (command, asynchronous, pipe_in, pipe_out, else exec_result = last_command_exit_value; + + if (user_subshell && was_error_trap && ignore_return == 0 && invert == 0 && exec_result != EXECUTION_SUCCESS) + { + last_command_exit_value = exec_result; + run_error_trap (); + } + + if (user_subshell && ignore_return == 0 && invert == 0 && exit_immediately_on_error && exec_result != EXECUTION_SUCCESS) + { +fprintf(stderr, "user subshell errexit\n"); + last_command_exit_value = exec_result; + run_pending_traps (); + jump_to_top_level (ERREXIT); + } + return (last_command_exit_value = exec_result); } else @@ -741,10 +762,8 @@ execute_command_internal (command, asynchronous, pipe_in, pipe_out, } } - /* 10/6/2008 -- added test for pipe_in and pipe_out because they indicate - the presence of a pipeline, and (until Posix changes things), a - pipeline failure should not cause the parent shell to exit on an - unsuccessful return status, even in the presence of errexit.. */ + /* 2009/02/13 -- pipeline failure is processed elsewhere. This handles + only the failure of a simple command. */ if (was_error_trap && ignore_return == 0 && invert == 0 && pipe_in == NO_PIPE && pipe_out == NO_PIPE && exec_result != EXECUTION_SUCCESS) { last_command_exit_value = exec_result; @@ -1056,8 +1075,11 @@ print_formatted_time (fp, format, rs, rsf, us, usf, ss, ssf, cpu) else if (s[1] == 'P') { s++; +#if 0 + /* clamp CPU usage at 100% */ if (cpu > 10000) cpu = 10000; +#endif sum = cpu / 100; sum_frac = (cpu % 100) * 10; len = mkfmt (ts, 2, 0, sum, sum_frac); @@ -2083,7 +2105,7 @@ execute_connection (command, asynchronous, pipe_in, pipe_out, fds_to_close) { REDIRECT *rp; COMMAND *tc, *second; - int ignore_return, exec_result; + int ignore_return, exec_result, was_error_trap, invert; ignore_return = (command->flags & CMD_IGNORE_RETURN) != 0; @@ -2149,7 +2171,25 @@ execute_connection (command, asynchronous, pipe_in, pipe_out, fds_to_close) break; case '|': + was_error_trap = signal_is_trapped (ERROR_TRAP) && signal_is_ignored (ERROR_TRAP) == 0; + invert = (command->flags & CMD_INVERT_RETURN) != 0; + ignore_return = (command->flags & CMD_IGNORE_RETURN) != 0; + exec_result = execute_pipeline (command, asynchronous, pipe_in, pipe_out, fds_to_close); + + if (was_error_trap && ignore_return == 0 && invert == 0 && exec_result != EXECUTION_SUCCESS) + { + last_command_exit_value = exec_result; + run_error_trap (); + } + + if (ignore_return == 0 && invert == 0 && exit_immediately_on_error && exec_result != EXECUTION_SUCCESS) + { + last_command_exit_value = exec_result; + run_pending_traps (); + jump_to_top_level (ERREXIT); + } + break; case AND_AND: |