From 06f3b2e6799ebd66a813ce9345c2ca6bbd0abe7d Mon Sep 17 00:00:00 2001
From: Brian Paul <brianp@vmware.com>
Date: Thu, 22 Jan 2009 15:06:17 -0700
Subject: 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().
---
 src/mesa/main/fbobject.c | 10 ++++++++++
 1 file changed, 10 insertions(+)

(limited to 'src/mesa')

diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index faf3763b71..f904937078 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) {
-- 
cgit v1.2.3