なぜscanfループからセグメンテーションフォルトが発生しますか？

Question

scanfループをコメントアウトすると、セグメント化エラーが消えます。ループが私にセグメンテーションフォールトを与えているのはなぜですか？

char** nameArray = malloc(numNames * sizeof(char *)); 

for(i =0; i< 10; i++) { 
    nameArray[i] = malloc(25 * sizeof(char)); 
} 

for (i = 0; i< numNames; i++) { 
    scanf("%s", &nameArray[i]); 
} 

for (i =0; i<numNames; i++) { 
    free(nameArray[i]); 
} 

Answer 1

まず、あなたは十分なエントリを作成する必要があるとして、

for(i =0; i< numNames; i++) { 

に

for(i =0; i< 10; i++) { 

を変更する必要があります。

はまた、あなたは、必要に応じて文字ポインタであるnameArray[i]として

scanf("%s", nameArray[i]); 

にこのライン

scanf("%s", &nameArray[i]); 

を変更する必要があります。

また、バッファオーバーランを防止するであろう。このよう

scanf("%24s", nameArray[i]); 

を使う方が良いでしょう。また、nameArray[i]はすでにmallocから入手したポインタ、であるので、それがあるので、あなただけの

scanf("%s", nameArray[i]); 

を使用して、scanfにそのポインタを渡す必要が乏しい

Answer 2

の戻り値をチェックすることをお勧めだろう&なし。

Answer 3

あなたは割り当てをしなければなりません。 あなたのケースであなただけの10 namearray要素

を割り当てるあなたはscanfのだろう、すべての変数の割り当てを行う必要があります。 numNames

for(i =0; i< numNames; i++) { 
    nameArray[i] = malloc(25 * sizeof(char)); 
} 

変化10と&記号がポインタの場合にはノルマルcharlsで使用されていますそれを使わないでください。

scanf("%s",nameArray[i]);