github-mirror/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell.git synced 2025-02-19 07:13:35 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix type of pid_status variable

Browse Source 
We had pid_status defined as a pid_t instance, which was fine since on
most platforms pid_t is an alias for int. However, that is not
universally the case and waitpid takes an int *, not a pid_t *.
This commit is contained in:
Mahmoud Al-Qudsi 2017-09-26 08:16:36 -05:00
parent 3604522bf2
commit 9fdfe44236
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/exec.cpp
View File

@ -1115,8 +1115,8 @@ void exec_job(parser_t &parser, job_t *j) {
// child_set_group() at this point. but we only need to call set_child_group for the
// first process in the group. If needs_keepalive is set, this has already been called
// for the keepalive process.
pid_t pid_status{};
int result;
int pid_status;
while ((result = waitpid(p->pid, &pid_status, WUNTRACED)) == -1 && errno == EINTR) {
// This could be a superfluous interrupt or Ctrl+C at the terminal In all cases, it
// is OK to retry since the forking code above is specifically designed to never,