将字符串复制到数组中——strcpy segfault

Question

我正在尝试将一个字符串复制到一个数组并打印它。它适用于第一个 for 循环，但第二次出现 seg 错误。

main (int argc, char *argv[]){
 int argcIndex;
 char **argumentArray = NULL;

 for(argcIndex=0; argcIndex < argc; argcIndex++){
     argumentArray = (char**) malloc((argc+1)*sizeof(char*));
     argumentArray[argcIndex] = (char*) malloc(((strlen(argv[argcIndex])+1)*sizeof(char)));
     strcpy(argumentArray[argcIndex], argv[argcIndex]); //Works here
     printf("argumentArray[%d]: %s \n", argcIndex, argumentArray[argcIndex]); //Works here
 }

 for(argcIndex=0; argcIndex < argc; argcIndex++){
     //strcpy(argumentArray[argcIndex], argv[argcIndex]); //This gives me a segfault
     printf("argumentArray[%d]: %s \n", argcIndex, argumentArray[argcIndex]); //This will only grab the last string in array
 }

}

Answer 1

你需要移动argumentArray的分配

 argumentArray = (char**) malloc((argc+1)*sizeof(char*));

在第一个 for 循环之外，否则，每次迭代，您都会覆盖（最终泄漏）内存。

下面的问题是，malloc() 返回未初始化的内存，而对于最后一次迭代，只有一个索引，argc-1 正在写入，所以其他 index 仍然不确定。在后面的循环中，当您尝试使用该值时，它会调用 undefined behavior。

也就是说，

• sizeof(char) 在 C 中保证为 1。乘以它没有任何意义。
• Please see this discussion on why not to cast the return value of malloc() and family in C.。

• 如果可用，您可以使用strdup()，而不是malloc() 和strcpy()。

  注意：strdup() 不是标准 C 函数，FWIW。