added inline environment

corrected env -i crash & SHLVL

Makefile

@@ -6,7 +6,7 @@
 #    By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+         #
 #                                                 +#+#+#+#+#+   +#+            #
 #    Created: 2019/03/27 16:51:02 by tmaze             #+#    #+#              #
-#    Updated: 2020/01/22 17:50:24 by tmaze            ###   ########.fr        #
+#    Updated: 2020/01/30 12:00:42 by tmaze            ###   ########.fr        #
 #                                                                              #
 #******************************************************************************#
 
@@ -27,7 +27,7 @@ endif
 
 #	Compilator
 CC		=	gcc
-FLAGS	=	-Wall -Wextra -Werror -g -fsanitize=address
+FLAGS	=	-Wall -Wextra -Werror
 
 #	Folders
 LIBDIR	=	libft
@@ -43,7 +43,8 @@ SRC	=	main.c			\
 		cmd_cd.c		\
 		cmd_echo.c		\
 		ms_exec.c		\
-		ms_ext.c
+		ms_ext.c		\
+		put_error_cd.c
 
 OBJ		=	$(SRC:.c=.o)
 

includes/minishell.h

@@ -6,7 +6,7 @@
 /*   By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2018/11/18 13:12:34 by tmaze             #+#    #+#             */
-/*   Updated: 2019/12/06 09:54:47 by tmaze            ###   ########.fr       */
+/*   Updated: 2020/01/26 22:19:25 by tmaze            ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -34,5 +34,9 @@ int					cmd_echo(char **argv, t_env **env);
 char				**res_ext(char **argv, t_env *env);
 
 int					exec_cmd(char **argv, t_env **env);
+void				*ft_strdel_null(char **s);
+
+void				*put_error_cd(char *file, char *msg);
+int					put_error_cd2(char *file, char *msg);
 
 #endif

libft/Makefile

@@ -6,7 +6,7 @@
 #    By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+         #
 #                                                 +#+#+#+#+#+   +#+            #
 #    Created: 2018/04/07 12:47:06 by tmaze             #+#    #+#              #
-#    Updated: 2019/12/06 09:59:05 by tmaze            ###   ########.fr        #
+#    Updated: 2020/01/25 16:24:08 by tmaze            ###   ########.fr        #
 #                                                                              #
 #******************************************************************************#
 

libft/srcs/ft_envtochar.c

@@ -17,7 +17,7 @@ char	*ft_envtochar(t_env *env)
 	int		size;
 	char	*ret;
 
-	size = ft_strlen(env->key) + ft_strlen(env->val) + 1;
+	size = ft_strlen(env->key) + ft_strlen(env->val) + 2;
 	if ((ret = ft_strnew(size)) != NULL)
 	{
 		ft_strlcpy(ret, env->key, size);

srcs/cmd_cd.c

@@ -6,20 +6,12 @@
 /*   By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2019/01/07 16:44:40 by tmaze             #+#    #+#             */
-/*   Updated: 2019/11/11 01:09:36 by tmaze            ###   ########.fr       */
+/*   Updated: 2020/01/26 22:17:48 by tmaze            ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
 #include "minishell.h"
 
-void	put_error_cd(char *file, char *msg)
-{
-	ft_putstr("cd: ");
-	ft_putstr(file);
-	ft_putstr(": ");
-	ft_putendl(msg);
-}
-
 char	*check_path_slash_cd(char *exec)
 {
 	int			i;
@@ -29,22 +21,15 @@ char	*check_path_slash_cd(char *exec)
 	if (exec[0] == '/')
 	{
 		if ((i = access(exec, F_OK)) != 0)
-			put_error_cd(exec, "no such file or directory");
+			return (put_error_cd(exec, "no such file or directory"));
-		if (i != 0)
-			return (NULL);
 		if ((i = lstat(exec, &info)) != 0)
-			put_error_cd(exec, "can't determine info");
+			return (put_error_cd(exec, "can't determine info"));
-		if (i != 0)
+		if ((S_ISLNK(info.st_mode)
-			return (NULL);
+			&& ((i = stat(exec, &info2)) != 0 || !S_ISDIR(info2.st_mode)))
-		if (!S_ISDIR(info.st_mode) || (S_ISLNK(info.st_mode) && ((i = stat(exec, &info2)) != 0 || !S_ISDIR(info2.st_mode))))
+			&& !S_ISDIR(info.st_mode))
-		{
+			return (put_error_cd(exec, "not a directory"));
-			put_error_cd(exec, "not a directory");
-			return (NULL);
-		}
 		if ((i = access(exec, X_OK)) != 0)
-			put_error_cd(exec, "permission denied");
+			return (put_error_cd(exec, "permission denied"));
-		if (i != 0)
-			return (NULL);
 		return (exec);
 	}
 	return (NULL);
@@ -59,23 +44,16 @@ int		cmd_cd_update_env(char *path, t_env **env, char opt)
 
 	ft_bzero(p_path, 4096);
 	if (!(pwd = ft_envgetelem("PWD", *env)))
-	{
+		return (put_error_cd2(path, "env var PWD undefined"));
-		put_error_cd(path, "env var PWD undefined");
-		return (1);
-	}
 	if (getcwd(p_path, 4096) == NULL || (oldpwd = ft_strdup(pwd->val)) == NULL
 		|| ft_realpath(path, &c_path) == NULL)
-	{
+		return (put_error_cd2(path, "error"));
-		put_error_cd(path, "error");
-		return (1);
-	}
 	if (ft_envupdate("PWD", *env, ((opt == 'P') ? p_path : c_path)) == NULL
 		|| ft_envupdate("OLDPWD", *env, oldpwd) == NULL)
 	{
-		put_error_cd(path, "error");
 		ft_strdel(&oldpwd);
 		ft_strdel(&c_path);
-		return (1);
+		return (put_error_cd2(path, "error"));
 	}
 	ft_strdel(&oldpwd);
 	ft_strdel(&c_path);
@@ -122,35 +100,42 @@ char	cd_getparams(char **argv, size_t *i)
 	return (ret[0]);
 }
 
-int		cmd_cd(char **argv, t_env **env)
+int		cmd_cd_switchboard(char **av, t_env **env, char opt, int i)
 {
-	t_env	*elem;
+	t_env	*e;
-	size_t	i;
+	char	*p;
-	char	*path;
-	char	opt;
 	int		ret;
 
-	i = 0;
+	if ((!av[i] && (e = ft_envgetelem("HOME", *env)) != NULL && e->val
-	if ((opt = cd_getparams(argv, &i)) == '\0')
+		!= NULL) || (av[i] && av[i][0] == '-'
-		return (1);
+		&& (e = ft_envgetelem("OLDPWD", *env)) != NULL))
-	if (!argv[i] && (elem = ft_envgetelem("HOME", *env)) != NULL && elem->val != NULL)
+		return (cmd_cd_core(e->val, env, opt));
-		return (cmd_cd_core(elem->val, env, opt));
+	else if (av[i] && av[i][0] == '/')
-	else if (argv[i] && argv[i][0] == '/')
+		return (cmd_cd_core(av[i], env, opt));
-		return (cmd_cd_core(argv[i], env, opt));
+	else if (av[i] && av[i][0] != '/'
-	else if (argv[i] && argv[i][0] == '-' && (elem = ft_envgetelem("OLDPWD", *env)) != NULL)
+		&& (e = ft_envgetelem("PWD", *env)) != NULL && e->val != NULL)
-		return (cmd_cd_core(elem->val, env, opt));
-	else if (argv[i] && argv[i][0] != '/' && (elem = ft_envgetelem("PWD", *env)) != NULL && elem->val != NULL)
 	{
-		if ((path = ft_strnew(ft_strlen(elem->val) + ft_strlen(argv[i]) + 1)) == NULL)
+		if ((p = ft_strnew(ft_strlen(e->val) + ft_strlen(av[i]) + 1)) == NULL)
-			put_error_cd(argv[i], "memory error");
+			put_error_cd(av[i], "memory error");
-		if (path == NULL)
+		if (p == NULL)
 			return (1);
-		ft_strcpy(path, elem->val);
+		ft_strcpy(p, e->val);
-		path[ft_strlen(path)] = '/';
+		p[ft_strlen(p)] = '/';
-		ft_strcat(path, argv[i]);
+		ft_strcat(p, av[i]);
-		ret = cmd_cd_core(path, env, opt);
+		ret = cmd_cd_core(p, env, opt);
-		ft_strdel(&path);
+		ft_strdel(&p);
 		return (ret);
 	}
 	return (1);
 }
+
+int		cmd_cd(char **argv, t_env **env)
+{
+	size_t	i;
+	char	opt;
+
+	i = 0;
+	if ((opt = cd_getparams(argv, &i)) == '\0')
+		return (1);
+	return (cmd_cd_switchboard(argv, env, opt, i));
+}

srcs/cmd_env.c

@@ -6,7 +6,7 @@
 /*   By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2019/09/20 14:33:48 by tmaze             #+#    #+#             */
-/*   Updated: 2019/12/09 09:59:58 by tmaze            ###   ########.fr       */
+/*   Updated: 2020/01/30 15:41:08 by tmaze            ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -47,6 +47,24 @@ int			env_cleanup(t_env *env_cp, int code)
 	return (code);
 }
 
+t_env		*inline_env(char **argv, int *i, t_env **env_cp)
+{
+	t_env	*new;
+
+	while (ft_strchr(argv[*i], '='))
+	{
+		if ((new = ft_envnew(argv[*i])) == NULL)
+		{
+			ft_printf("env: memory error\n");
+			ft_envdel(env_cp);
+			break ;
+		}
+		ft_envaddend(env_cp, new);
+		(*i)++;
+	}
+	return (*env_cp);
+}
+
 int			cmd_env(char **argv, t_env **env)
 {
 	int		i;
@@ -55,8 +73,7 @@ int			cmd_env(char **argv, t_env **env)
 	t_env	*env_cp;
 
 	i = 1;
-	env_cp = NULL;
+	if (!(env_cp = NULL) && !ft_strequ(argv[i], "-i") && (env != NULL))
-	if (!ft_strequ(argv[i], "-i") && (env != NULL))
 	{
 		it = *env;
 		while (it)
@@ -67,8 +84,9 @@ int			cmd_env(char **argv, t_env **env)
 			it = it->next;
 		}
 	}
-	if (ft_strequ(argv[i], "-i"))
+	else if (ft_strequ(argv[i], "-i"))
 		i++;
+	inline_env(argv, &i, &env_cp);
 	if (argv[i])
 		exec_cmd(argv + i, &env_cp);
 	else

srcs/main.c

@@ -6,12 +6,51 @@
 /*   By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2019/09/19 17:08:46 by tmaze             #+#    #+#             */
-/*   Updated: 2019/11/10 21:24:36 by tmaze            ###   ########.fr       */
+/*   Updated: 2020/01/30 13:47:35 by tmaze            ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
 #include "minishell.h"
 
+t_env	*def_env(void)
+{
+	t_env	*ret;
+	t_env	*n;
+	char	p[PATH_MAX];
+
+	n = ft_memalloc(sizeof(t_env));
+	if (n != NULL && ((n->key = ft_strnew(3)) == NULL || !getcwd(p, PATH_MAX)))
+		ft_envdelelem(&n);
+	if (n != NULL)
+		ft_strcpy(n->key, "PWD");
+	if (n != NULL && (n->val = ft_strdup(p)) == NULL)
+		ft_envdelelem(&n);
+	if (n != NULL)
+		ft_envaddend(&ret, n);
+	n = ft_memalloc(sizeof(t_env));
+	if (n != NULL && (n->key = ft_strnew(5)) == NULL)
+		ft_envdelelem(&n);
+	if (n != NULL)
+		ft_strcpy(n->key, "SHLVL");
+	if (n != NULL && (n->val = ft_strdup("1")) == NULL)
+		ft_envdelelem(&n);
+	if (n != NULL)
+		ft_envaddend(&ret, n);
+	return (ret);
+}
+
+char	*inc_shlvl(char **shlvl)
+{
+	char	*ret;
+
+	if ((ret = ft_itoa(ft_atoi(*shlvl) + 1)) != NULL)
+	{
+		ft_strdel(shlvl);
+		*shlvl = ret;
+	}
+	return (ret);
+}
+
 t_env	*env2lst(char **env)
 {
 	t_env	*ret;
@@ -23,7 +62,11 @@ t_env	*env2lst(char **env)
 	while (env[i])
 	{
 		if ((new = ft_envnew(env[i])) != NULL)
+		{
+			if (ft_strequ(new->key, "SHLVL") && ft_str_is_numeric(new->val))
+				inc_shlvl(&new->val);
 			ft_envaddend(&ret, new);
+		}
 		else
 		{
 			ft_envdelelem(&new);
@@ -32,58 +75,60 @@ t_env	*env2lst(char **env)
 		}
 		i++;
 	}
+	if (i == 0 && ret == NULL && (ret = def_env()) == NULL)
+		ft_printf("Env: Memory Error\n");
 	return (ret);
 }
 
-int		main(void)
+void	check_exec(char **argv, t_env *env)
 {
-	extern char			**environ;
+	int					i;
 	static t_builtin	built[S_BIN] = {{"env", &cmd_env}, {"cd", &cmd_cd}
 								, {"setenv", &cmd_setenv}, {"echo", &cmd_echo}
 								, {"unsetenv", &cmd_unsetenv}};
-	char				**argv;
-	t_env				*env;
-	char				*cmd;
-	int					i;
 
-	env = NULL;
+	i = -1;
-	if ((env = env2lst(environ)) == NULL)
+	while (++i < S_BIN)
-		return (2);
-	while (1)
-	{
-		cmd = NULL;
-		ft_printf("%s$>%s ", FT_COLOR_GREEN, FT_RESET);
-		if (ft_getline(&cmd) >= 0 && cmd != NULL
-			&& (argv = ft_strsplit(cmd, ' ')) != NULL)
-		{
-			ft_strdel(&cmd);
-			if ((res_ext(argv, env)) != NULL)
-			{
-				if (ft_strequ(argv[0], "exit"))
-				{
-					ft_del_words_tables(&argv);
-					ft_envdel(&env);
-					return (0);
-				}
-				i = 0;
-				while (i < S_BIN)
-				{
 		if (ft_strequ(argv[0], built[i].cmd))
 		{
 			built[i].f(argv, &env);
 			break ;
 		}
-					i++;
-				}
 	if (i == S_BIN)
 		exec_cmd(argv, &env);
-			}
+}
-		}
+
-		ft_strdel(&cmd);
+int		cleanup(char **argv, t_env *env)
+{
 	ft_del_words_tables(&argv);
-	}
-	ft_del_words_tables(&argv);
-	ft_strdel(&cmd);
 	ft_envdel(&env);
 	return (0);
 }
+
+int		main(void)
+{
+	extern char	**environ;
+	char		**argv;
+	t_env		*env;
+	char		*cmd;
+
+	if ((env = env2lst(environ)) == NULL)
+		return (2);
+	while (1)
+	{
+		ft_printf("%s$>%s ", FT_COLOR_GREEN, FT_RESET);
+		if (ft_getline(&cmd) >= 0 && cmd != NULL
+			&& (argv = ft_strsplit(cmd, ' ')) != NULL && !ft_strdel_null(&cmd))
+		{
+			if ((res_ext(argv, env)) != NULL)
+			{
+				if (ft_strequ(argv[0], "exit"))
+					return (cleanup(argv, env));
+				check_exec(argv, env);
+			}
+		}
+		ft_strdel(&cmd);
+		ft_del_words_tables(&argv);
+	}
+	return (0);
+}

srcs/ms_exec.c

@@ -6,7 +6,7 @@
 /*   By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2019/09/26 16:25:00 by tmaze             #+#    #+#             */
-/*   Updated: 2019/11/10 15:52:54 by tmaze            ###   ########.fr       */
+/*   Updated: 2020/01/30 11:48:25 by tmaze            ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -23,12 +23,13 @@ int		ft_isin(char *str, char c)
 	return (-1);
 }
 
-void	put_error(char *file, char *msg)
+void	*put_error(char *file, char *msg)
 {
 	ft_putstr("minishell: ");
 	ft_putstr(file);
 	ft_putstr(": ");
 	ft_putendl(msg);
+	return (NULL);
 }
 
 char	*check_path_slash(char *exec)
@@ -64,10 +65,7 @@ char	*check_path_dot(char *exec, t_env *env)
 
 	if ((path = ft_envgetelem("PWD", env)) == NULL
 		|| path->val == NULL || (tmp = ft_strjoin(path->val, "/")) == NULL)
-	{
+		return (put_error(exec, "memory error"));
-		put_error(exec, "memory error");
-		return (NULL);
-	}
 	if ((ret = ft_strjoin(tmp, exec)) == NULL)
 		put_error(exec, "memory error");
 	ft_strdel(&tmp);
@@ -77,21 +75,53 @@ char	*check_path_dot(char *exec, t_env *env)
 	{
 		if (access(ret, X_OK) == 0)
 			return (exec);
-		put_error(exec, "permission denied");
 		ft_strdel(&ret);
-		return (NULL);
+		return (put_error(exec, "permission denied"));
 	}
-	put_error(exec, "no such file or directory");
 	ft_strdel(&ret);
+	return (put_error(exec, "no such file or directory"));
+}
+
+void	*ft_strdel_null(char **s)
+{
+	ft_strdel(s);
 	return (NULL);
 }
 
+char	*check_path_elems(char **path_elems, char *exec)
+{
+	int		i;
+	char	*tmp;
+	char	*ret;
+
+	i = -1;
+	ret = NULL;
+	while (path_elems[++i])
+	{
+		if ((tmp = ft_strjoin(path_elems[i], "/")) == NULL
+			&& !put_error(exec, "memory error"))
+			break ;
+		if ((ret = ft_strjoin(tmp, exec)) == NULL
+			&& !put_error(exec, "memory error") && !ft_strdel_null(&tmp))
+			break ;
+		if (!ft_strdel_null(&tmp) && access(ret, F_OK) == 0)
+		{
+			if (access(ret, X_OK) == 0)
+				break ;
+			ft_strdel(&ret);
+			ft_del_words_tables(&path_elems);
+			return (put_error(exec, "permission denied"));
+		}
+		ft_strdel(&ret);
+	}
+	return (ret);
+}
+
 char	*check_path(char *exec, t_env *env)
 {
 	size_t	i;
 	t_env	*path;
 	char	**path_elems;
-	char	*tmp;
 	char	*ret;
 
 	ret = NULL;
@@ -104,43 +134,29 @@ char	*check_path(char *exec, t_env *env)
 		if ((path = ft_envgetelem("PATH", env)) == NULL
 			|| path->val == NULL
 			|| (path_elems = ft_strsplit(path->val, ':')) == NULL)
-		{
+			return (put_error(exec, "could not resolve path"));
-			put_error(exec, "could not resolve path");
-			return (NULL);
-		}
 		i = 0;
-		while (path_elems[i])
+		if ((ret = check_path_elems(path_elems, exec)) == NULL)
 		{
-			if ((tmp = ft_strjoin(path_elems[i], "/")) == NULL)
-				put_error(exec, "memory error");
-			if (tmp == NULL)
-				break ;
-			if ((ret = ft_strjoin(tmp, exec)) == NULL)
-			{
-				put_error(exec, "memory error");
-				ft_strdel(&tmp);
-				break ;
-			}
-			ft_strdel(&tmp);
-			if (access(ret, F_OK) == 0)
-			{
-				if (access(ret, X_OK) == 0)
-					break ;
-				put_error(exec, "permission denied");
-				ft_strdel(&ret);
 			ft_del_words_tables(&path_elems);
-				return (NULL);
+			return (put_error(exec, "command not found"));
 		}
-			ft_strdel(&ret);
-			i++;
-		}
-		if (path_elems[i] == NULL)
-			put_error(exec, "command not found");
 		ft_del_words_tables(&path_elems);
 	}
 	return (ret);
 }
 
+void	exec_cmd_child(char *path, char **argv, char **env_tab, t_env **env)
+{
+	execve(path, argv, env_tab);
+	ft_putendl_fd("minishell: error", 1);
+	if (ft_strcmp(path, argv[0]) != 0)
+		ft_strdel(&path);
+	ft_del_words_tables(&env_tab);
+	ft_envdel(env);
+	exit(-1);
+}
+
 int		exec_cmd(char **argv, t_env **env)
 {
 	int		ret;
@@ -151,19 +167,10 @@ int		exec_cmd(char **argv, t_env **env)
 		return (-1);
 	if ((path = check_path(argv[0], *env)) == NULL)
 		ft_del_words_tables(&env_tab);
-	ft_printf("path after resolve: %s\n", path);
 	if (path == NULL)
 		return (-1);
 	if ((ret = fork()) == 0)
-	{
+		exec_cmd_child(path, argv, env_tab, env);
-		execve(path, argv, env_tab);
-		ft_putendl_fd("minishell: error", 1);
-		if (ft_strcmp(path, argv[0]) != 0)
-			ft_strdel(&path);
-		ft_del_words_tables(&env_tab);
-		ft_envdel(env);
-		exit(-1);
-	}
 	else if (ret == -1)
 		ft_putendl_fd("minishell: error", 1);
 	if (ret == -1)

srcs/ms_ext.c

@@ -6,7 +6,7 @@
 /*   By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+        */
 /*                                                +#+#+#+#+#+   +#+           */
 /*   Created: 2019/10/25 18:21:47 by tmaze             #+#    #+#             */
-/*   Updated: 2020/01/23 00:23:24 by tmaze            ###   ########.fr       */
+/*   Updated: 2020/01/30 15:42:17 by tmaze            ###   ########.fr       */
 /*                                                                            */
 /* ************************************************************************** */
 
@@ -18,6 +18,14 @@ void	*return_null(char *msg)
 	return (NULL);
 }
 
+void	ft_strreplace(char **dst, char *src, int *index, t_env *elem)
+{
+	if (*dst != NULL)
+		ft_strdel(dst);
+	*dst = src;
+	*index = ft_strlen(elem->val);
+}
+
 t_env	*search_key(t_env *env, char *cmd, int index)
 {
 	t_env	*it;
@@ -37,10 +45,7 @@ t_env	*search_key(t_env *env, char *cmd, int index)
 			j++;
 		}
 		if (!it->key[i] && (!ret || ft_strlen(it->key) > ft_strlen(ret->key)))
-		{
-			ft_printf("found key: %s\n", it->key);
 			ret = it;
-		}
 		it = it->next;
 	}
 	return (ret);
@@ -52,11 +57,12 @@ char	*replace_env(char *src, t_env *elem, int key_size, int i)
 	int		res_len;
 
 	res_len = ft_strlen(src) - key_size + ft_strlen(elem->val);
+	ft_printf("size replace: %d\n", res_len);
 	if ((ret = ft_strnew(res_len)) != NULL)
 	{
-		ft_strlcpy(ret, src, i + 1);
+		ft_strncpy(ret, src, i);
 		ft_strlcat(ret, elem->val, res_len + 1);
-		ft_strlcat(ret, src + 1 + key_size, res_len + 1);
+		ft_strlcat(ret, src + key_size, res_len + 1);
 	}
 	else
 		ft_printf("minishell: memory error\n");
@@ -73,9 +79,7 @@ char	*resolve_complete_key(char **src, int *index, t_env *env)
 		if ((ret = replace_env(*src, tmp, ft_strlen(tmp->key)
 			+ 1, *index)) != NULL)
 		{
-			ft_strdel(src);
+			ft_strreplace(src, ret, index, tmp);
-			*src = ret;
-			*index += ft_strlen(tmp->val);
 			return (*src);
 		}
 		else
@@ -91,25 +95,23 @@ char	**res_ext(char **av, t_env *env)
 	t_env	*tmp;
 	char	*ret;
 
-	i[0] = 0;
+	i[0] = -1;
-	while (av && av[i[0]] && !(i[1] = 0))
+	while (av && av[++i[0]] && !(i[1] = 0))
 	{
-		while (av[i[0]] && av[i[0]][i[1]])
+		while (av[i[0]][i[1]])
 		{
-			if (av[i[0]][i[1]] == '~' && (tmp = ft_envgetelem("HOME", env))
+			if (av[i[0]][i[1]] != '$' && av[i[0]][i[1]] != '~')
+				i[1]++;
+			else if (av[i[0]][i[1]] == '~' && (tmp = ft_envgetelem("HOME", env))
 				!= NULL && (ret = replace_env(av[i[0]], tmp, 1, i[1])) != NULL)
-			{
+				ft_strreplace(&av[i[0]], ret, &i[1], tmp);
-				ft_strdel(&av[i[0]]);
+			else if (av[i[0]][i[1]] == '~' && (tmp = ft_envgetelem("HOME", env))
-				av[i[0]] = ret;
+				== NULL)
-				i[1] += ft_strlen(tmp->val);
+				return (return_null("Entry HOME not found\n"));
-			}
 			else if (av[i[0]][i[1]] == '$'
 				&& resolve_complete_key(&av[i[0]], &i[1], env) == NULL)
 				return (NULL);
-			else if (av[i[0]][i[1]] != '$' && av[i[0]][i[1]] == '~')
-				i[1]++;
 		}
-		i[0]++;
 	}
 	return (av);
 }

srcs/put_error_cd.c

@@ -0,0 +1,31 @@
+/* ************************************************************************** */
+/*                                                                            */
+/*                                                        :::      ::::::::   */
+/*   put_error_cd.c                                     :+:      :+:    :+:   */
+/*                                                    +:+ +:+         +:+     */
+/*   By: tmaze <tmaze@student.42.fr>                +#+  +:+       +#+        */
+/*                                                +#+#+#+#+#+   +#+           */
+/*   Created: 2020/01/26 22:18:09 by tmaze             #+#    #+#             */
+/*   Updated: 2020/01/26 22:18:24 by tmaze            ###   ########.fr       */
+/*                                                                            */
+/* ************************************************************************** */
+
+#include "minishell.h"
+
+void	*put_error_cd(char *file, char *msg)
+{
+	ft_putstr("cd: ");
+	ft_putstr(file);
+	ft_putstr(": ");
+	ft_putendl(msg);
+	return (NULL);
+}
+
+int		put_error_cd2(char *file, char *msg)
+{
+	ft_putstr("cd: ");
+	ft_putstr(file);
+	ft_putstr(": ");
+	ft_putendl(msg);
+	return (1);
+}