glGetError: 0x502が発生する原因が知りたい。

Question

提示コードですがcamera->RendererBuffer()部で下記のエラー出ますこれは何が原因なのでしょうか？

現状

Camera->RendererBuffer()をコメントアウトすると普通に動くのですが他のクラスでこの関数を使うとエラーにはなりません。
これは何が影響しているのでしょうか？

glGetError: https://qiita.com/_ydah/items/da56763e94ba58af3d91

0x502: 現在のステートで無効な操作をしている場合、もしくは廃止された関数を呼び出した場合(*1)

利用ライブラリ:
opengl
freetype
glew
glfw
stbi

ソースコード全文: https://github.com/Shigurechan/AAEditor/tree/e0db42df3945fc1e1ea9879e2746ffe5eeedf9f2

Camera

// ##################################### 初期化　##################################### 
FrameWork::Camera::Camera()
{	

	// レンダリングポリゴン
	glGenVertexArrays(1, &quadVAO);
	glGenBuffers(1, &quadVBO);

	shader = std::make_shared<FrameWork::Shader>();

	shader->Load("Asset/shader/FrameBuffer.vert", "Asset/shader/FrameBuffer.frag");

	shader->setEnable();
	glBindVertexArray(quadVAO);
	glBindBuffer(GL_ARRAY_BUFFER, quadVBO);

	glBufferData(GL_ARRAY_BUFFER, 24 * sizeof(float), &quad, GL_STATIC_DRAW);

	GLuint attrib = shader->getAttribLocation("vertexPosition");
	glEnableVertexAttribArray(attrib);
	glVertexAttribPointer(attrib, 2, GL_FLOAT, GL_FALSE, 4 * sizeof(float), (void*)0);
	shader->setBindAttribLocation("vertexPosition");

	attrib = shader->getAttribLocation("vertexUV");
	glEnableVertexAttribArray(attrib);
	glVertexAttribPointer(attrib, 2, GL_FLOAT, GL_FALSE, 4 * sizeof(float), (void*)(2 * sizeof(float)));
	shader->setBindAttribLocation("vertexUV");

	shader->setDisable();
	glBindVertexArray(0);
	glBindBuffer(GL_ARRAY_BUFFER, 0);

	

	//MSAA バッファ
	glGenFramebuffers(1, &frameBufferFBO);	
	glBindFramebuffer(GL_FRAMEBUFFER, frameBufferFBO);	
	glGenTextures(1, &frameBuffer);
	glBindTexture(GL_TEXTURE_2D_MULTISAMPLE, frameBuffer);
	glTexImage2DMultisample(GL_TEXTURE_2D_MULTISAMPLE, 8, GL_RGB, windowContext->getSize().x, windowContext->getSize().y, GL_TRUE);
	glFramebufferTexture2D(GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, GL_TEXTURE_2D_MULTISAMPLE, frameBuffer, 0);
	

	//レンダリングバッファ
	glGenRenderbuffers(1, &rbo);
	glBindRenderbuffer(GL_RENDERBUFFER, rbo);
	glRenderbufferStorageMultisample(GL_RENDERBUFFER, 8, GL_DEPTH24_STENCIL8, windowContext->getSize().x, windowContext->getSize().y);
	glFramebufferRenderbuffer(GL_FRAMEBUFFER, GL_DEPTH_STENCIL_ATTACHMENT, GL_RENDERBUFFER, rbo);

	//MSAA　アタッチ
	GLuint attachment = GL_COLOR_ATTACHMENT0;
	glDrawBuffers(1,&attachment);
	

	if (glCheckFramebufferStatus(GL_FRAMEBUFFER) != GL_FRAMEBUFFER_COMPLETE)
	{
		std::cerr << "Framebuffer is not complete: 0x" << glCheckFramebufferStatus(GL_FRAMEBUFFER) << std::endl;
		assert(0);
	}
	glBindFramebuffer(GL_FRAMEBUFFER,0);


	//深度マップ
	glGenFramebuffers(1, &depthMapFBO);
	glBindFramebuffer(GL_FRAMEBUFFER, depthMapFBO);
	glGenTextures(1, &depthMap);
	glBindTexture(GL_TEXTURE_2D, depthMap);
	glTexImage2D(GL_TEXTURE_2D, 0, GL_DEPTH_COMPONENT, windowContext->getSize().x, windowContext->getSize().y, 0, GL_DEPTH_COMPONENT, GL_FLOAT, NULL);
	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_NEAREST);
	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_REPEAT);
	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_REPEAT);
	glFramebufferTexture2D(GL_FRAMEBUFFER, GL_DEPTH_ATTACHMENT, GL_TEXTURE_2D, depthMap, 0);

	//カラーバッファ
	glGenFramebuffers(1, &colorBufferFBO);
	glBindFramebuffer(GL_FRAMEBUFFER, colorBufferFBO);
	glGenTextures(1, &colorBuffer);
	glBindTexture(GL_TEXTURE_2D, colorBuffer);
	glTexImage2D(GL_TEXTURE_2D, 0, GL_RGB, windowContext->getSize().x, windowContext->getSize().y, 0, GL_RGB, GL_UNSIGNED_BYTE, NULL);
	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
	glFramebufferTexture2D(GL_FRAMEBUFFER, GL_COLOR_ATTACHMENT0, GL_TEXTURE_2D, colorBuffer, 0);	// we only need a color buffer
	



	position = glm::vec3(0,0,0.5);	//座標
	vecLook = glm::vec3(0,0,-1);	//向き

	setPosition(position);
	setLook(vecLook);

}


// ##################################### スカイボックスを設定　##################################### 
void FrameWork::Camera::setSkyBox(GLuint tex)
{

}


// ##################################### 描画　##################################### 
void FrameWork::Camera::View(std::shared_ptr<Actor> actor)
{

	glBindFramebuffer(GL_FRAMEBUFFER, depthMapFBO);
	glViewport(0, 0, windowContext->getSize().x, windowContext->getSize().y);

	//actor->Renderer_Depth(getViewPerspective());

	glBindFramebuffer(GL_FRAMEBUFFER, frameBufferFBO);
	glViewport(0, 0, windowContext->getSize().x, windowContext->getSize().y);

	glEnable(GL_DEPTH_TEST);
//	actor->Renderer(getViewPerspective());

	glDepthFunc(GL_LEQUAL);

	actor->Renderer(getViewOrthographic());


//	actor->Renderer(getViewPerspective());
	glDepthFunc(GL_LESS);

	glBindFramebuffer(GL_FRAMEBUFFER,0);
}

// ##################################### フレームバッファ　クリア　##################################### 
void FrameWork::Camera::ClearBuffer()
{
	glEnable(GL_DEPTH_TEST);
	glClearColor(0.0f, 0.0f, 0.0f, 1.0f);
	glBindFramebuffer(GL_FRAMEBUFFER, depthMapFBO);
	glClear(GL_DEPTH_BUFFER_BIT);
	glBindFramebuffer(GL_FRAMEBUFFER, frameBufferFBO);
	glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);
	//glBindFramebuffer(GL_FRAMEBUFFER, 0);
}

// ##################################### フレームバッファ　描画　##################################### 
void FrameWork::Camera::RendererBuffer()
{

	glClear(GL_COLOR_BUFFER_BIT | GL_DEPTH_BUFFER_BIT);

	glBindVertexArray(quadVAO);
	glBindBuffer(GL_ARRAY_BUFFER, quadVBO);
	shader->setEnable();
	
	glBindFramebuffer(GL_READ_FRAMEBUFFER, frameBuffer);
	glBindFramebuffer(GL_DRAW_FRAMEBUFFER, colorBufferFBO);
	glBlitFramebuffer(0, 0, windowContext->getSize().x, windowContext->getSize().y, 0, 0, windowContext->getSize().x, windowContext->getSize().y, GL_COLOR_BUFFER_BIT, GL_NEAREST);
	

	glBindFramebuffer(GL_FRAMEBUFFER, 0);
	shader->setUniformSampler2D("screenTexture", 0,colorBufferFBO);
	glDrawArrays(GL_TRIANGLES,0,6);

	shader->setDisable();	
	glBindVertexArray(0);
	glBindBuffer(GL_ARRAY_BUFFER, 0);
	
}

Title.hpp

#include "Title.hpp"
#include <FrameWork.hpp>
#include "Title_Menu.hpp"

Title::Title() : Scene(Scene::SceneType::Title)
{
	title = std::make_shared<Title_Menu>();
	camera = std::make_unique<FrameWork::Camera>();

}



void Title::Update()
{
	//title->Update();
	




}

void Title::Renderer()
{
	camera->ClearBuffer();

	camera->View(title);


	camera->RendererBuffer();

}


void Title::Loop()
{
	Update();
	Renderer();
}

Title_menu

#include "Title_Menu.hpp"
#include "Layer.hpp"

#include <FrameWork.hpp>

Title_Menu::Title_Menu() : Actor()
{
	renderer = std::make_shared<FrameWork::Text>();
	layer = std::make_shared<Layer>(glm::ivec2(18,32),25);

	//layer->setCharacter(glm::ivec2(0,0),'A');

}



void Title_Menu::Update()
{








}

void Title_Menu::Renderer(const glm::mat4 view)const
{
	/*
	for (int y = 0; y < layer->getSize().y; y++)
	{
		for (int x = 0; x < layer->getSize().x; x++)
		{
			renderer->Renderer(view,glm::ivec2(x * 25, y * 25),layer->getCharacter(glm::ivec2(x,y)).getTexture());
		}
	}

	*/
}

Layer

#include "Layer.hpp"
#include <FrameWork.hpp>

Layer::Layer(glm::ivec2 s,int ps)
{	
	position = glm::ivec2(0,ps);	//座標
	pixelSize = ps;					//ピクセルサイズ
	size = glm::vec2(0,0);			//サイズ


	character.resize(0);

	Resize(s);						//リサイズ
}


void Layer::setCharacter(glm::ivec2 pos, const char ch)
{
	character.at(pos.y).at(pos.x).setCharacter(ch);
}

void Layer::Resize(glm::ivec2 s)
{
	if ((s.x > size.x) && (s.y > size.y))
	{
		for (int y = 0; y < s.y - size.y; y++)
		{
			std::vector<Character> ch;
			for (int x = 0; x < s.x - size.x; x++)
			{
				ch.push_back(Character(glm::ivec2(size.x + x, size.y + y),pixelSize));
			}
			character.push_back(ch);
		}

		for (int y = size.y; y < s.y; y++)
		{
			for (int x = size.x; x < s.x; x++)
			{
				character.at(y).at(x).setCharacter(' ');
			}
		}
	}



	size = s;
}

void Layer::setPixelSize(int s)
{
	for (int y = 0; y < size.y; y++)
	{
		for (int x = 0; x < size.x; x++)
		{
			character.at(y).at(x).Resize(s);
		}
	}
}

Character Layer::getCharacter(glm::ivec2 pos)
{
	return character.at(pos.y).at(pos.x);
}

glm::ivec2 Layer::getSize()
{
	return size;
}


void Layer::Update()
{

}

Chracter

#include "Character.hpp"

Character::Character(glm::vec2 pos,int pixel)
{
	position = pos;
	character = '\0';	
	pixelSize = pixel;
}

void Character::setCharacter(const char ch)
{
	character = ch;
	DeleteCharacterTexture(texture);
	texture = FrameWork::GenCharacterTexture("Asset/font/PressStart2P.ttf", pixelSize, FrameWork::GetRGBAColor(glm::vec4(0, 255, 0, 255)), "%c",ch);
}

void Character::Resize(int pixel)
{
	pixelSize = pixel;
	DeleteCharacterTexture(texture);
	texture = FrameWork::GenCharacterTexture("Asset/font/PressStart2P.ttf", pixelSize, FrameWork::GetRGBAColor(glm::vec4(0, 255, 0, 255)), "%c", character);
}



glm::ivec2 Character::getPosition()
{
	return position;
}

FrameWork::CharacterTexture* Character::getTexture()
{
	return &texture;
}

void Character::Update()
{
	
}

void Character::Renderer(const glm::ivec2 pos,const glm::mat4 view)const
{
	switch(character)
	{

	case 0x28:	// (
	{

	}
	break;

	case 0x29:	// )
	{

	}
	break;


	default:
	{

	}
	break;
		
	}		
}

Answer 1

std::vector<Character> ch;
for (int x = 0; x < s.x - size.x; x++)
{
	ch.push_back(Character(glm::ivec2(size.x + x, size.y + y),pixelSize));

Character はコピー・ムーブに正しく対応できている必要があります。Characterクラス内で何かしらのリソースの確保・開放処理がある場合はかなり注意深く実装する必要があります。
対処方法はこれまで4回ぐらい書いた気がします。

Comments

[退会済みユーザー]

すいません。よく考えましたが対処法がわかりません。このクラスにはアドレス変数を等は入っていないので
一時的変数のアドレスがコピーされて無効なアドレスが変数に入っているとは思えないのですが

[int32_t]

アドレスは関係ありません。
Character::texture はインスタンスがコピーされると不味いように見えますよ。

https://teratail.com/questions/318749
https://teratail.com/questions/318818

[退会済みユーザー]

textureは構造体です。またアドレス変数は持っていないのですが texture変数はテクスチャの札のようなものなので
コピーされても大丈夫だと思うのですが何がおかしいのでしょうか？　もう少し詳細によろしいでしょうか？

[int32_t]

アドレスは関係ありません。
Character がコピーされると Character::texture::textureID の開放処理が複数回走ってしまってまずくないですか、ということです。

[退会済みユーザー]

なるほど。ベクター配列に要素を確保するfor文と初期値を設定するfor文とで分けたのですが同じエラー出ます。
これはなぜでしょうか？
またtextureIDの解放処理が毎回走るとありますがデストラクタは書いていません。これはどいう意味なのでしょうか？　追記提示コードを追加しました。

[int32_t]

完全なコードなしで推測で問題になりそうなことを指摘しているだけなので、この回答が的外れである可能性はもちろんあります。頑張ってデバッグしてくださいね。