From: Brian Paul <brianp@vmware.com>
Date: Thu, 22 Jan 2009 22:06:17 +0000 (-0700)
Subject: mesa: additional FBO/RB id error checking for ARB_fbo
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=06f3b2e6799ebd66a813ce9345c2ca6bbd0abe7d;p=mesa.git

mesa: additional FBO/RB id error checking for ARB_fbo

It's illegal to bind a FBO/RB id that didn't come from glGenRender/Framebuffer().
---

diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index faf3763b716..f9049370784 100644
--- a/src/mesa/main/fbobject.c
+++ b/src/mesa/main/fbobject.c
@@ -616,6 +616,11 @@ _mesa_BindRenderbufferEXT(GLenum target, GLuint renderbuffer)
          newRb = NULL;
       }
       if (!newRb) {
+         if (ctx->Extensions.ARB_framebuffer_object) {
+            /* All RB IDs must be Gen'd */
+	    _mesa_error(ctx, GL_INVALID_OPERATION, "glBindRenderbuffer(buffer)");
+	    return;
+         }
 	 /* create new renderbuffer object */
 	 newRb = ctx->Driver.NewRenderbuffer(ctx, renderbuffer);
 	 if (!newRb) {
@@ -1018,6 +1023,11 @@ _mesa_BindFramebufferEXT(GLenum target, GLuint framebuffer)
          newFb = NULL;
       }
       if (!newFb) {
+         if (ctx->Extensions.ARB_framebuffer_object) {
+            /* All FBO IDs must be Gen'd */
+	    _mesa_error(ctx, GL_INVALID_OPERATION, "glBindFramebuffer(buffer)");
+	    return;
+         }
 	 /* create new framebuffer object */
 	 newFb = ctx->Driver.NewFramebuffer(ctx, framebuffer);
 	 if (!newFb) {