Renderbuffer allocation fix for GLES3

src/OpenGL/libGLESv2/Context.cpp

@@ -1266,6 +1266,8 @@ void Context::bindDrawFramebuffer(GLuint framebuffer)
 
 void Context::bindRenderbuffer(GLuint renderbuffer)
 {
+	mResourceManager->checkRenderbufferAllocation(renderbuffer);
+
     mState.renderbuffer = getRenderbuffer(renderbuffer);
 }
 

src/OpenGL/libGLESv2/ResourceManager.cpp

@@ -324,12 +324,6 @@ Renderbuffer *ResourceManager::getRenderbuffer(unsigned int handle)
     }
     else
     {
-		if (!renderbuffer->second)
-		{
-			Renderbuffer *renderbufferObject = new Renderbuffer(handle, new Colorbuffer(0, 0, GL_RGBA4_OES, 0));
-			mRenderbufferMap[handle] = renderbufferObject;
-			renderbufferObject->addRef();
-		}
         return renderbuffer->second;
     }
 }
@@ -414,6 +408,16 @@ void ResourceManager::checkTextureAllocation(GLuint texture, TextureType type)
     }
 }
 
+void ResourceManager::checkRenderbufferAllocation(GLuint handle)
+{
+	if(handle != 0 && !getRenderbuffer(handle))
+	{
+		Renderbuffer *renderbufferObject = new Renderbuffer(handle, new Colorbuffer(0, 0, GL_RGBA4_OES, 0));
+		mRenderbufferMap[handle] = renderbufferObject;
+		renderbufferObject->addRef();
+	}
+}
+
 void ResourceManager::checkSamplerAllocation(GLuint sampler)
 {
 	if(sampler != 0 && !getSampler(sampler))

src/OpenGL/libGLESv2/ResourceManager.h

@@ -80,6 +80,7 @@ class ResourceManager
 
     void checkBufferAllocation(unsigned int buffer);
     void checkTextureAllocation(GLuint texture, TextureType type);
+    void checkRenderbufferAllocation(GLuint handle);
     void checkSamplerAllocation(GLuint sampler);
 
     bool isSampler(GLuint sampler);

src/OpenGL/libGLESv2/libGLESv2.cpp

@@ -509,10 +509,12 @@ void BindRenderbuffer(GLenum target, GLuint renderbuffer)
 
 	if(context)
 	{
-		if(renderbuffer != 0 && !context->getRenderbuffer(renderbuffer))
+		// [OpenGL ES 3.0.4] Section 4.4.2 page 204
+		// If renderbuffer is not zero, then the resulting renderbuffer object
+		// is a new state vector, initialized with a zero-sized memory buffer.
+		if(renderbuffer != 0 && !context->getRenderbuffer(renderbuffer) && (context->getClientVersion() < 3))
 		{
 			// [OpenGL ES 2.0.25] Section 4.4.3 page 112
-			// [OpenGL ES 3.0.2] Section 4.4.2 page 201
 			// 'renderbuffer' must be either zero or the name of an existing renderbuffer object of
 			// type 'renderbuffertarget', otherwise an INVALID_OPERATION error is generated.
 			return error(GL_INVALID_OPERATION);