mesa: fix the low limit of width and height for glRenderbufferStorage
authorYuanhan Liu <yuanhan.liu@linux.intel.com>
Tue, 25 Oct 2011 07:36:59 +0000 (15:36 +0800)
committerYuanhan Liu <yuanhan.liu@linux.intel.com>
Thu, 3 Nov 2011 02:22:56 +0000 (10:22 +0800)
glRenderbufferStorage man page says:

  GL_INVALID_VALUE is generated if either of width or height is negative,
  or greater than the value of GL_MAX_RENDERBUFFER_SIZE.

NOTE: this is a candidate for the 7.11 branch

Signed-off-by: Yuanhan Liu <yuanhan.liu@linux.intel.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
src/mesa/main/fbobject.c

index c56062ac6a7402b79e9977ebce495a4c9f516f08..ff46570bf16c3b18a6e16a7475771ed28da4add2 100644 (file)
@@ -1370,12 +1370,12 @@ renderbuffer_storage(GLenum target, GLenum internalFormat,
       return;
    }
 
-   if (width < 1 || width > (GLsizei) ctx->Const.MaxRenderbufferSize) {
+   if (width < 0 || width > (GLsizei) ctx->Const.MaxRenderbufferSize) {
       _mesa_error(ctx, GL_INVALID_VALUE, "%s(width)", func);
       return;
    }
 
-   if (height < 1 || height > (GLsizei) ctx->Const.MaxRenderbufferSize) {
+   if (height < 0 || height > (GLsizei) ctx->Const.MaxRenderbufferSize) {
       _mesa_error(ctx, GL_INVALID_VALUE, "%s(height)", func);
       return;
    }