文字列を配列にコピーする - strcpy segfault

Question

文字列を配列にコピーして印刷しようとしています。最初のforループでは動作しますが、2回目にsegフォルトが発生します。あなたがそうでない場合は、全ての反復、あなたが上書きしている（そして最後に、漏れ）メモリ、最初forループの外側argumentArray

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

のための割り当てを移動する必要があり

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

。その下に

問題は、初期化されていないメモリmalloc()リターンであり、最後の反復のために、唯一の指標、argc-1は書き込まを取得しているので、他のインデックスの内容は不定のままです。後のループで、その値を使用しようとすると、undefined behaviorが呼び出されます。

• sizeof(char)言っ

  は、それによって何かを掛けるにはポイントがありませんCで1であることが保証されます。

• Please see this discussion on why not to cast the return value of malloc() and family in C.。

• の代わりにmalloc()とstrcpy()の場合は、strdup()を使用できます（利用可能な場合）。

  注：strdup()は標準のC関数FWIWではありません。