メモリーリークと思われるがどうしても場所がわからない。原因と場所。

Question

提示コードの一番下のところでブレークポイントが発生してしまいます。また提示画像にもある通り文章を日本語にすると

「CRT detected that the application wrote to memory after end of heap buffer」
日本語 「CRTは、ヒープバッファーの終了後にアプリケーションがメモリに書き込んだことを検出しました」
と表示されています。一体どこでメモリリークが発生しているのでしょうか？ またマイクロソフトのサイトは専門的過ぎてわかりません。
malloc();してある場所だと思い自分でいろいろprintf();を置いてどこまで実行出来ているか探りましたがわかりません

参考サイト: https://so-zou.jp/software/tech/programming/cpp/run-time-library/memory/
参考サイト: https://docs.microsoft.com/ja-jp/previous-versions/technical-document/ms810466(v=msdn.10)?redirectedfrom=MSDN

//シェーダー関係のログを取得
void Game::GetShader_Log(GLuint shader_type, GLuint Log_type)
{
	GLint bufferSize = 0;
	GLchar* infoLog = nullptr;

	//成功したかどうか？
	glGetShaderiv(shader_type, Log_type, &bufferSize);
	

	//ログの長さを取得する
	glGetShaderiv(shader_type, GL_INFO_LOG_LENGTH, &bufferSize);
	
	infoLog = (GLchar*)malloc(sizeof(bufferSize + 10));
	
	if (infoLog != NULL)
	{
		GLsizei length;//ログの長さ	
		glGetShaderInfoLog(shader_type, bufferSize, &length, infoLog);

		if (length > 1)//null文字を含むため一文字以上の場合ログ出力
		{
			fprintf(stderr, "InfoLog:%s\n\n", infoLog);

		}
		printf("ああああ\n");

		free(infoLog);
		infoLog = NULL;
		printf("いいいいいい\n");


	}

	//return true;
}

Comments

[dodox86]

> infoLog = (GLchar*)malloc(sizeof(bufferSize + 10));

の、sizeof(bufferSize + 10) は何なのでしょう。意図通りですか？　もう少し丁寧にコーディングしましょう。

[退会済みユーザー]

なんとんなくもう少し多くメモリを確保したら溢れないのかなと思いましたがこれはそれとは別に悪い行為なのでしょうか？

[dodox86]

いやそういうことではなく、そもそもsizeof(bufferSize + 10) とは何に対するsizeof演算子か？と言うことです。bufferSizeはGLint型であり、更に +10 に対するsizeofって？？？何？　です。

[退会済みユーザー]

なるほど。sizeof(bufferSize);に変更しました。また誤字だと思われますがGLintですよね？

[dodox86]

(GLintの誤記は修正済みです）　sizeof(bufferSize)じゃダメでしょう。それは実質sizeof(GLint) です。

[退会済みユーザー]

自己解決しました。回答をご覧ください。

Answer 1

日本語 「CRTは、ヒープバッファーの終了後にアプリケーションがメモリに書き込んだことを検出しました」

それはメモリリークとは別物です。メモリリークは、「使いもしないメモリを確保したまま放置状態となっている」という事象です。

Comments

[退会済みユーザー]

そうですかしかしちゃんとfree();しているはずなのですがなぜでしょうか？

Answer 2

infoLog = (GLchar*)malloc(sizeof(bufferSize + 10));

コードを

infoLog = (GLchar*)malloc(bufferSize);

に変更したら治りました。sizeofはその変数のメモリの数字が返ってくるので違いますよね
その変数の値がそのバイトなのでGLintつまり４バイトのメモリが返ってきますが文字列は可変長なので
必ず４バイトとも限りませんよってこれは違います。buferSizeだけでいいのですその中の数字が重要なので