Valgrind недействительным чтения размера 1 на execvp

Question

Я использую execvp для мини-проекта оболочки и Valgrind показывает мне огромную утечку памяти на одной линии: линию 263.

Я проверил, что я не я не использую ни инициализированные переменные, и я освобождаю свои указатели и вкладки. Так что я не сейчас, где искать ...

Кто-нибудь видит, в чем проблема?

173 char **token_tab(char *line) 
174 { 
175 char **res_tab = NULL; 
176 token *tok_tab = NULL; 
177 token *tmp = NULL; 
178 int cpt = 0; 
179 
180 tok_tab = generate_token(line); 
181 tmp = tok_tab; 
182 
183 while (tmp != NULL) 
184 { 
185  cpt++; 
186  tmp = tmp->next; 
187 } 
188 
189 res_tab = malloc(sizeof (char *) * cpt); 
190 
191 tmp = tok_tab; 
192 
193 for (int i = 0; i < cpt; ++i) 
194 { 
195  res_tab[i] = malloc(sizeof (char) * my_strlen(tmp->word)); 
196  my_strcpy(res_tab[i], tmp->word); 
197  tmp = tmp->next; 
198 } 
199 res_tab[cpt - 1] = NULL; 
200 
201 return res_tab; 
202 } 


    int exec_cmd(char **tab_word, pid_t pid) 
259 { 
260 int status = -5; 
261 pid = fork(); 
262 if (pid == 0) 
263  execvp(tab_word[0], tab_word); 
264 else 
265  wait(&status); 
266 
267 free_double_char(tab_word); 
268 
269 return WEXITSTATUS(status); 
270 } 

    minishell$ls 
==22475== Conditional jump or move depends on uninitialised value(s) 
==22475== at 0x400E91: main (minishell.c:357) 
==22475== 
==22475== Conditional jump or move depends on uninitialised value(s) 
==22475== at 0x400E69: main (minishell.c:358) 
==22475== 
==22475== Conditional jump or move depends on uninitialised value(s) 
==22475== at 0x401177: generate_token (new_lex.c:38) 
==22475== by 0x400AA4: token_tab (minishell.c:180) 
==22475== by 0x400EA7: main (minishell.c:361) 
==22475== 
==22475== Invalid write of size 1 
==22475== at 0x402B6B: my_strcpy (my_strcpy.c:10) 
==22475== by 0x400B47: token_tab (minishell.c:196) 
==22475== by 0x400EA7: main (minishell.c:361) 
==22475== Address 0x51feb22 is 0 bytes after a block of size 2 alloc'd 
==22475== at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) 
==22475== by 0x400B1B: token_tab (minishell.c:195) 
==22475== by 0x400EA7: main (minishell.c:361) 
==22475== 
==22476== Invalid read of size 1 
==22476== at 0x4C2DBB0: __GI_strchr (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) 
==22476== by 0x4EF8978: execvpe (execvpe.c:60) 
==22476== by 0x400CB1: exec_cmd (minishell.c:263) 
==22476== by 0x400EDE: main (minishell.c:366) 
==22476== Address 0x51feb22 is 0 bytes after a block of size 2 alloc'd 
==22476== at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) 
==22476== by 0x400B1B: token_tab (minishell.c:195) 
==22476== by 0x400EA7: main (minishell.c:361) 
==22476== 
==22476== Invalid read of size 1 
==22476== at 0x4C2E0F4: strlen (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) 
==22476== by 0x4EF8AC4: execvpe (execvpe.c:101) 
==22476== by 0x400CB1: exec_cmd (minishell.c:263) 
==22476== by 0x400EDE: main (minishell.c:366) 
==22476== Address 0x51feb22 is 0 bytes after a block of size 2 alloc'd 
==22476== at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) 
==22476== by 0x400B1B: token_tab (minishell.c:195) 
==22476== by 0x400EA7: main (minishell.c:361) 
==22476== 
==22476== Invalid read of size 1 
==22476== at 0x4C2FF3E: __GI_memcpy (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) 
==22476== by 0x4EF8B1D: execvpe (execvpe.c:126) 
==22476== by 0x400CB1: exec_cmd (minishell.c:263) 
==22476== by 0x400EDE: main (minishell.c:366) 
==22476== Address 0x51feb22 is 0 bytes after a block of size 2 alloc'd 
==22476== at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) 
==22476== by 0x400B1B: token_tab (minishell.c:195) 
==22476== by 0x400EA7: main (minishell.c:361) 
==22476== 
==22476== Syscall param execve(argv[i]) points to unaddressable byte(s) 
==22476== at 0x4EF8337: execve (execve.c:33) 
==22476== by 0x4EF8B88: execvpe (execvpe.c:149) 
==22476== by 0x400CB1: exec_cmd (minishell.c:263) 
==22476== by 0x400EDE: main (minishell.c:366) 
==22476== Address 0x51feb22 is 0 bytes after a block of size 2 alloc'd 
==22476== at 0x4C2AB80: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so) 
==22476== by 0x400B1B: token_tab (minishell.c:195) 
==22476== by 0x400EA7: main (minishell.c:361) 
==22476== 

спасибо

Answer 1

В коде, вы не выделяя для последнего '\0' символа для строк.

Линия

res_tab[i] = malloc(sizeof (char) * my_strlen(tmp->word)); 

следует выделять больше 1 CHAR

res_tab[i] = malloc(sizeof (char) * (my_strlen(tmp->word) + 1)); 

Я предполагаю, что здесь, что my_strlen() похож на strlen() и не рассчитывать '\0' в длину.

Answer 2

У вас есть утечка памяти. Вы переписываете последний malloc на res_tab [cpt-1] с помощью NULL, не освобождая вначале. Если последний маркер не должен быть пропущен:

res_tab = malloc(sizeof (char *) * (cpt+1)); 

и

res_tab[cpt] = NULL;