diff options
| author | Charles Cabergs <me@cacharle.xyz> | 2020-09-13 11:34:26 +0200 |
|---|---|---|
| committer | Charles Cabergs <me@cacharle.xyz> | 2020-09-13 14:51:10 +0200 |
| commit | e63d7878ad5d5ad7ceec11e1e8ac79aced93fbc0 (patch) | |
| tree | 1cf021174499677c415eeda12bb9210758a661cf /src/eval/cmd.c | |
| parent | 7e7219da26b9d256e451f8dddb66641d11f75434 (diff) | |
| download | minishell-e63d7878ad5d5ad7ceec11e1e8ac79aced93fbc0.tar.gz minishell-e63d7878ad5d5ad7ceec11e1e8ac79aced93fbc0.tar.bz2 minishell-e63d7878ad5d5ad7ceec11e1e8ac79aced93fbc0.zip | |
Fixing pipe zombie children
Diffstat (limited to 'src/eval/cmd.c')
| -rw-r--r-- | src/eval/cmd.c | 14 |
1 files changed, 4 insertions, 10 deletions
diff --git a/src/eval/cmd.c b/src/eval/cmd.c index 9745632..82e30c9 100644 --- a/src/eval/cmd.c +++ b/src/eval/cmd.c @@ -6,7 +6,7 @@ /* By: charles <charles@student.42.fr> +#+ +:+ +#+ */ /* +#+#+#+#+#+ +#+ */ /* Created: 2020/06/14 10:41:31 by charles #+# #+# */ -/* Updated: 2020/09/12 17:04:23 by charles ### ########.fr */ +/* Updated: 2020/09/13 14:19:37 by charles ### ########.fr */ /* */ /* ************************************************************************** */ @@ -68,12 +68,6 @@ int forked_cmd(void *void_param) struct stat statbuf; param = void_param; - /* ft_vecpop(param->env_local, NULL); */ - /* if (ft_vecswallow_at(param->env, param->env->size - 1, param->env_local) == NULL) */ - /* { */ - /* ft_vecdestroy(param->env_local, free); */ - /* return (EVAL_FATAL); */ - /* } */ if (param->builtin != NULL) return (param->builtin->func(param->argv, param->env)); else @@ -96,7 +90,7 @@ int forked_cmd(void *void_param) } } -int eval_cmd(int fds[2], t_env env, t_path path, t_ast *ast) +int eval_cmd(int fds[2], t_env env, t_path path, t_ast *ast, pid_t *child_pid) { t_fork_param_cmd param; char **argv; @@ -124,12 +118,12 @@ int eval_cmd(int fds[2], t_env env, t_path path, t_ast *ast) return (127); } } - else if (!param.builtin->forked) + else if (!param.builtin->forked && child_pid == NULL) return (param.builtin->func(argv, env)); param.argv = argv; param.env = env; - status = fork_wrap(fds, ¶m, &forked_cmd, NULL); + status = fork_wrap(fds, ¶m, &forked_cmd, child_pid); ft_split_destroy(argv); g_last_status = status; return (status); |
