c: установить массив массива malloc'd в NULL (недействительная запись)

Question

Я работаю с параллельными процессами, и в одной части моего кода я хочу разблокировать новый процесс, который вызывает ту же программу с несколько разными аргументами. Например, если пользователь изначально называл «./prog arg1 arg2 arg3», я хотел бы разблокировать новый процесс, который выполняет «./prog -n 1 arg2 arg3», а затем эта программа будет работать с fork и exec »./prog - n 2 arg3. " Единственный способ, которым я могу это сделать, - использовать execv(), но я столкнулся с проблемами, потому что execv требует, чтобы последний элемент в массиве argv [], переданный ему, был NULL. Ниже мой код, и я опубликую результат valgrind ниже этого.

 //fork an additional process: ./prog -n (i+1) args... 
     // could accomplish by execv("./prog", paramlist), where paramlist is 
     //  the same as -n (i+1) and then argv, skipping argv[1] 

     //create new paramlist 
     char **params = malloc(sizeof(char*) * argc + 2); 

     sprintf(numbuf, "%d", i+1); 

     //./prog -n (i+1) argv[2:argc-1] 
     params[0] = argv[0]; 
     params[1] = "-n"; 
     params[2] = numbuf; 
     for(int i = 2; i<argc; i++){ //skip argv[0] bc we already assigned it 
      params[i+1] = argv[i];  //skip argv[1] because we just worked on it 

     } 
     params[argc+1] = NULL;   //list must be null terminated 

     pid3 = fork(); 
     if(0 == pid3){ 
      execv("./prog", params); 
      exit(-1); 
     } //no waitpid here, because we'd like this process to continue on its own 

     params[argc+1] = "";   //avoid invalid free 
     free(params); 

Когда я запускаю этот код нормально, я получаю следующее сообщение об ошибке:

*** Error in `./prog': free(): invalid next size (fast): 0x0000000001ac3830 *** 

Valgrind говорит мне это:

==28067== Memcheck, a memory error detector 
==28067== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al. 
==28067== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info 
==28067== Command: ./prog ./photos/photo1.jpg 
==28067== 
=====Displaying photo 0 (be patient.. XQuartz sucks) 
Enter degrees to rotate image(0,90,180,270): 90 
Enter a caption > thumb 
==28067== Invalid write of size 8 
==28067== at 0x400C58: main (in /usr/prog) 
==28067== Address 0x51fa8d0 is 16 bytes inside a block of size 18 alloc'd 
==28067== at 0x4C2BBAD: malloc (vg_replace_malloc.c:299) 
==28067== by 0x400C05: main (in /usr/prog) 
==28067== 
==28067== Invalid write of size 8 
==28067== at 0x400CC3: main (in /usr/prog) 
==28067== Address 0x51fa8d8 is 6 bytes after a block of size 18 alloc'd 
==28067== at 0x4C2BBAD: malloc (vg_replace_malloc.c:299) 
==28067== by 0x400C05: main (in /usr/prog) 
==28067== 
==28067== Invalid write of size 8 
==28067== at 0x400D0E: main (in /usr/prog) 
==28067== Address 0x51fa8d8 is 6 bytes after a block of size 18 alloc'd 
==28067== at 0x4C2BBAD: malloc (vg_replace_malloc.c:299) 
==28067== by 0x400C05: main (in /usr/prog) 
==28067== 
==28071== Syscall param execve(argv) points to unaddressable byte(s) 
==28071== at 0x4F00CF7: execve (in /usr/lib64/libc-2.23.so) 
==28071== by 0x400CE8: main (in /usr/prog) 
==28071== Address 0x51fa8d2 is 0 bytes after a block of size 18 alloc'd 
==28071== at 0x4C2BBAD: malloc (vg_replace_malloc.c:299) 
==28071== by 0x400C05: main (in /usr/prog) 
==28071== 
==28067== 
==28067== HEAP SUMMARY: 
==28067==  in use at exit: 0 bytes in 0 blocks 
==28067== total heap usage: 5 allocs, 5 frees, 1,051,194 bytes allocated 
==28067== 
==28067== All heap blocks were freed -- no leaks are possible 
==28067== 
==28067== For counts of detected and suppressed errors, rerun with: -v 
==28067== ERROR SUMMARY: 3 errors from 3 contexts (suppressed: 0 from 0) 

Интересно, если я закомментируйте «бесплатно (Params) ', то я получаю ошибку сегментации, возникающую на линии перед ней,' params [argc + 1] = ""; , Почему это может быть?

Answer 1

char **params = malloc(sizeof(char*) * argc + 2); 

Должно быть

char **params = malloc(sizeof(char*) * (argc + 2));