テキストファイルを逆順に出力

Question

###実現したいこと
mallocとreallocを使ってテキストファイルの記述を一行ずつ、逆順でもう一つのテキストファイルに出力させようとしているのですが、エラーがでて一向にできず困っています。
###発生している問題・エラーメッセージ

0xC0000005: 場所 0xCDCDCDCD の読み取り中にアクセス違反が発生しました。

###該当のソースコード

int main()
{
	FILE *fp_R;
	FILE *fp_W;
	char buff[256];
	char **ptr = (char**)malloc(0);
	fopen_s(&fp_R, "略", "r");
	fopen_s(&fp_W, "略", "w");

	if (fp_R == NULL) {
		puts("ファイルのオープンに失敗しました。");
		exit(-1);
	}

	if (fp_W == NULL) {
		puts("ファイルのオープンに失敗しました。");
		exit(-1);
	}

	int i = 0;
	while (fgets(buff, 100, fp_R) != NULL) {
		char *p = (char *)malloc(strlen(buff) + 1);
		strcpy_s(p, strlen(buff) + 1,buff);
		ptr[i] = p;
		++i;
		ptr = (char **)realloc(ptr, sizeof(char *)* i);
	}
	--i;
	while (i >= 0) {
		fprintf(fp_W,"%s\n",ptr[i]);
		--i;
	}

	free(ptr);

	fclose(fp_R);
	fclose(fp_W);
	return 0;

Answer 1

すでに解決済みになっていますが、おそらく解決していないと思われるので、詳しく説明しておきます。

ファイル読み込みループの中の

ptr[i] = p;
++i;
ptr = (char **)realloc(ptr, sizeof(char *)* i);

これだと領域の確保と利用の順番が逆になっています。確保してからでないと利用できません。
また、領域のサイズをsizeof(char *)* iとしていると、1件目、つまりi == 0のときにサイズが0となってしまうので、件数でサイズを計算するためにsizeof(char *) * (i + 1)としないといけません。

それらを踏まえて、このようにすれば良いでしょう。

ptr = (char **)realloc(ptr, sizeof(char *) * (i + 1));
ptr[i] = p;
++i;

それと、最初のptrの初期化に(char**)malloc(0)は無意味（にもかかわらず意味ありげに思えてしまい誤解を与える危険性がある）なので、単純にNULLで初期化すれば良いです。

Comments

[hatena777]

詳しく説明して頂きありがとうございました。とてもわかりやすくて、すぐに解決することができました。

Answer 2

領域を確保していないからです。

char ptr = (char)malloc(0);

reallocを使っているようですが、使う前に範囲外で落ちています。
また毎周reallocを使うのはちょっとコストが高すぎなんじゃないか、とも思います。

Comments

[hatena777]

mallocとrealloc覚えたてでよくわかっていませんでした。