diff options
-rw-r--r-- | cgit.h | 1 | ||||
-rw-r--r-- | shared.c | 7 |
2 files changed, 4 insertions, 4 deletions
@@ -62,7 +62,6 @@ struct cgit_filter { | |||
62 | int old_stdout; | 62 | int old_stdout; |
63 | int pipe_fh[2]; | 63 | int pipe_fh[2]; |
64 | int pid; | 64 | int pid; |
65 | int exitstatus; | ||
66 | }; | 65 | }; |
67 | 66 | ||
68 | struct cgit_repo { | 67 | struct cgit_repo { |
@@ -459,7 +459,6 @@ void cgit_prepare_repo_env(struct cgit_repo * repo) | |||
459 | 459 | ||
460 | int cgit_open_filter(struct cgit_filter *filter) | 460 | int cgit_open_filter(struct cgit_filter *filter) |
461 | { | 461 | { |
462 | |||
463 | filter->old_stdout = chk_positive(dup(STDOUT_FILENO), | 462 | filter->old_stdout = chk_positive(dup(STDOUT_FILENO), |
464 | "Unable to duplicate STDOUT"); | 463 | "Unable to duplicate STDOUT"); |
465 | chk_zero(pipe(filter->pipe_fh), "Unable to create pipe to subprocess"); | 464 | chk_zero(pipe(filter->pipe_fh), "Unable to create pipe to subprocess"); |
@@ -480,13 +479,15 @@ int cgit_open_filter(struct cgit_filter *filter) | |||
480 | 479 | ||
481 | int cgit_close_filter(struct cgit_filter *filter) | 480 | int cgit_close_filter(struct cgit_filter *filter) |
482 | { | 481 | { |
482 | int exit_status; | ||
483 | |||
483 | chk_non_negative(dup2(filter->old_stdout, STDOUT_FILENO), | 484 | chk_non_negative(dup2(filter->old_stdout, STDOUT_FILENO), |
484 | "Unable to restore STDOUT"); | 485 | "Unable to restore STDOUT"); |
485 | close(filter->old_stdout); | 486 | close(filter->old_stdout); |
486 | if (filter->pid < 0) | 487 | if (filter->pid < 0) |
487 | return 0; | 488 | return 0; |
488 | waitpid(filter->pid, &filter->exitstatus, 0); | 489 | waitpid(filter->pid, &exit_status, 0); |
489 | if (WIFEXITED(filter->exitstatus) && !WEXITSTATUS(filter->exitstatus)) | 490 | if (WIFEXITED(exit_status) && !WEXITSTATUS(exit_status)) |
490 | return 0; | 491 | return 0; |
491 | die("Subprocess %s exited abnormally", filter->cmd); | 492 | die("Subprocess %s exited abnormally", filter->cmd); |
492 | } | 493 | } |