セグメンテーションフォールト・ブロック

Question

FILE *fp; 

fp = fopen(filen, "wb"); 

const char tok[2] = ","; 
char str[340]; 

while (fgets(str, 340, stdin) != NULL) 
{ 
    struct test loadTest; 
    printf("You entered: %s", str); 
    strncpy(loadTest.level, strtok(str, tok), 20); 
    strncpy(loadTest.first, strtok(NULL, tok), 30); 
    fwrite(&loadTest, sizeof(struct test), 1, fp); 
} 

fclose(fp); 

すべてのこんにちは、私は私のコードでセグメンテーションフォールトエラーを取得していますいくつかの理由

。

上記の小さなコードブロックのどこかにエラーがあります（これはsegフォルトのために変更したものです）が、私はそれを特定することはできません。

セグメンテーションの不具合は私がアクセスすべきではないメモリへのアクセスと関係していることは知っていますが、コード内でどこでそれをしているのかわかりません。

ご協力いただければ幸いです！

Answer 1

ファイルへの読み取り/書き込みの前にあなたのコードのいくつかの改善

• fopenのチェック結果は、それらの値
• 使用sizeofの代わりに、一定の（コメントで述べたように）
を使用する前に、変数を初期化します
• strtok()はNULLを返すことができます。これはチェックする必要があります（hereを参照）
• あなたトンので、ここではthis

の慎重strncpy()を使用するには、コードのバージョン

FILE *fp; 

fp = fopen(filen, "wb"); 

if (fp == NULL) 
{ 
    printf("Error opening file: %s", filen); 
} 
else 
{ 

    const char tok[2] = ","; 
    char str[340]; 

    while (fgets(str, sizeof(str), stdin) != NULL) 
    { 
     struct test loadTest; 
     char *level; 
     char *first; 
     memset(&loadTest, 0, sizeof(loadTest)); 
     printf("You entered: %s", str); 
     level = strtok(str, tok); 
     if (level == NULL) 
     { 
      continue; // bad input ? 
     } 
     first = strtok(NULL, tok); 
     if (first == NULL) 
     { 
      continue; 
     } 
     strncpy(loadTest.level, level, sizeof(loadTest.level)-sizeof(char)); 
     strncpy(loadTest.first, first, sizeof(loadTest.first)-sizeof(char)); 
     fwrite(&loadTest, sizeof(loadTest), 1, fp); 
    } 

    fclose(fp); 
}