c言語 fcloseのfree()エラーに関する質問

Question

c言語のfcloseにて、突然エラーが発生するようになってしまい困っております。

エラーが起こっている部分はcsvファイルを読み込む関数です
101行以下の、縦に1列ごとに並んだ数字のデータが入ったcsvファイルを読み込み、
引数で渡された配列に代入するという関数です。

void read_csv(double *list, char *fname){
	FILE *fp;
	double f;
	int count = 0;
	fp = fopen(fname, "r");
	if(fp == NULL) {
		printf("%s file not open!\n", fname);
		//return -1;
	}
	while(fscanf(fp, "%lf", &f) != EOF) {
        list[count] = f;
        count ++;
        if(count == 101) break;
	}
	fclose(fp); // ファイルを閉じる
}

この関数を実行すると、fclose部分にて、

free(): invalid pointer
中止 (コアダンプ)

というエラーが発生するようになってしまいました。
今まで問題なく正常に動いていたのですが、ある時からこのエラーが出るようになってしまいました。
csvファイルの名前や中身は変えておりません。

理由、解決策がわかる方がいらっしゃればご教授いただきたいです。
よろしくお願いします。

Comments

[episteme]

read_csv() を呼んでる側のコードみせて。

[y_huto]

大変申し訳ございません。この部分しかお見せできないです。

Answer 1

問題は何かがメモリを破壊していることだと思います。

まず怪しまれるのは、listが正しいメモリ領域を指していないか、指しているメモリ領域に必要な容量が確保されていないのではないかということです。
確認してください。

それ以外のプログラムのコードに原因がある可能性もありますが、提示されている材料からはわかりません。

Comments

[y_huto]

ありがとうございました。理由はわからないのですが、関数を呼び出す直前のlistのサイズが9(正しい)、関数内でのサイズがなぜか1(誤り)になっていました。

[ikapy]

「今まで問題なく正常に動いていた」というのはたまたまコアダンプしなかっただけで、元々バグのあるシステムだったということです。呼び元のソース公開不可ということなので、なんとも言えませんが、count == 101でbreakですが、これ違和感あります。なんとも言えませんが101では100ではと・・・。呼び元の領域は101個とってますか。101個とるのは？ですね。
とは言え、他にも潜在バグがありそうなのでテストしなおした方がよいかも。テスト仕様書とエビデンス込みで。
大きなお世話でした。