42_adjoly/ft_minipowershell
1
0
Fork 0
mirror of https://github.com/KeyZox71/ft_minipowershell.git synced 2025-03-15 11:26:51 +01:00
Code Issues Actions Packages Projects Releases Wiki Activity

「🔨」 fix(parsing/split_cmd): fixed leak when no command

Browse Source
This commit is contained in:
y-syo
2024-07-21 17:47:38 +02:00
parent 27bda16995
commit 7483029244
4 changed files with 12 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
libft/src/mem/ft_free.c
View File

@ -6,7 +6,7 @@
/* By: mmoussou <marvin@42.fr> +#+ +:+ +#+ */ /* By: mmoussou <marvin@42.fr> +#+ +:+ +#+ */
/* +#+#+#+#+#+ +#+ */ /* +#+#+#+#+#+ +#+ */
/* Created: 2024/01/12 19:27:31 by mmoussou #+# #+# */ /* Created: 2024/01/12 19:27:31 by mmoussou #+# #+# */
/* Updated: 2024/01/18 01:05:44 by mmoussou ### ########.fr */ /* Updated: 2024/07/21 17:38:12 by mmoussou ### ########.fr */
/* */ /* */
/* ************************************************************************** */ /* ************************************************************************** */
@ -39,10 +39,13 @@ void free_tab(char ***arr)
int i; int i;
i = 0; i = 0;
while ((*arr)[i]) if (*arr)
{ {
free((*arr)[i]); while ((*arr)[i])
i++; {
free((*arr)[i]);
i++;
}
} }
free(*arr); free(*arr);
*arr = NULL; *arr = NULL;

4
src/exec/exec_split_cmd.c
View File

@ -6,7 +6,7 @@
/* By: mmoussou <mmoussou@student.42angouleme.fr +#+ +:+ +#+ */ /* By: mmoussou <mmoussou@student.42angouleme.fr +#+ +:+ +#+ */
/* +#+#+#+#+#+ +#+ */ /* +#+#+#+#+#+ +#+ */
/* Created: 2024/06/01 14:55:06 by mmoussou #+# #+# */ /* Created: 2024/06/01 14:55:06 by mmoussou #+# #+# */
/* Updated: 2024/07/21 12:58:49 by mmoussou ### ########.fr */ /* Updated: 2024/07/21 17:38:50 by mmoussou ### ########.fr */
/* */ /* */
/* ************************************************************************** */ /* ************************************************************************** */
@ -54,7 +54,7 @@ int exec_single_cmd(t_cmd *cmd, char **env, t_env *env_t, int pipe_fd[2])
if (!cmd->cmd) if (!cmd->cmd)
{ {
free(cmd->argv); ft_free("a", &(cmd->argv));
get_exit_code(0); get_exit_code(0);
return (0); return (0);
} }

2
src/parsing/split_argv.c
View File

@ -6,7 +6,7 @@
/* By: adjoly <adjoly@student.42angouleme.fr> +#+ +:+ +#+ */ /* By: adjoly <adjoly@student.42angouleme.fr> +#+ +:+ +#+ */
/* +#+#+#+#+#+ +#+ */ /* +#+#+#+#+#+ +#+ */
/* Created: 2024/06/05 13:48:57 by adjoly #+# #+# */ /* Created: 2024/06/05 13:48:57 by adjoly #+# #+# */
/* Updated: 2024/07/18 17:15:48 by adjoly ### ########.fr */ /* Updated: 2024/07/21 17:33:54 by mmoussou ### ########.fr */
/* */ /* */
/* ************************************************************************** */ /* ************************************************************************** */

4
src/parsing/split_cmd.c
View File

@ -6,7 +6,7 @@
/* By: mmoussou <mmoussou@student.42angouleme.fr +#+ +:+ +#+ */ /* By: mmoussou <mmoussou@student.42angouleme.fr +#+ +:+ +#+ */
/* +#+#+#+#+#+ +#+ */ /* +#+#+#+#+#+ +#+ */
/* Created: 2024/05/04 15:00:32 by adjoly #+# #+# */ /* Created: 2024/05/04 15:00:32 by adjoly #+# #+# */
/* Updated: 2024/07/20 17:49:22 by adjoly ### ########.fr */ /* Updated: 2024/07/21 17:44:29 by mmoussou ### ########.fr */
/* */ /* */
/* ************************************************************************** */ /* ************************************************************************** */
@ -24,9 +24,9 @@ t_cmd *split_cmd(char *cmd_av, t_cmd *cmd)
cmd->cmd = NULL; cmd->cmd = NULL;
return (cmd); return (cmd);
} }
cmd->argv = split;
cmd->cmd = ft_strdup(*split); cmd->cmd = ft_strdup(*split);
if (!cmd->cmd) if (!cmd->cmd)
return (cmd); return (cmd);
cmd->argv = split;
return (cmd); return (cmd);
} }