Mesa (master): mesa: don' t error check the default buffer object in glBindBufferOffsetEXT()

[Samuel Pitoiset]

Module: Mesa
Branch: master
Commit: 08ee28b6a86ff85eb1d4ac4b72be2aaf07f83729
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=08ee28b6a86ff85eb1d4ac4b72be2aaf07f83729

Author: Samuel Pitoiset <samuel.pitoi...@gmail.com>
Date:   Thu Aug 24 13:44:21 2017 +0200

mesa: don't error check the default buffer object in glBindBufferOffsetEXT()

An allocation check is already done when the buffer is created at
context creation.

Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com>
Reviewed-by: Timothy Arceri <tarc...@itsqueeze.com>

---

 src/mesa/main/transformfeedback.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/mesa/main/transformfeedback.c 
b/src/mesa/main/transformfeedback.c
index 07a9f9e940..a075d0875a 100644
--- a/src/mesa/main/transformfeedback.c
+++ b/src/mesa/main/transformfeedback.c
@@ -778,12 +778,11 @@ _mesa_BindBufferOffsetEXT(GLenum target, GLuint index, 
GLuint buffer,
       bufObj = ctx->Shared->NullBufferObj;
    } else {
       bufObj = _mesa_lookup_bufferobj(ctx, buffer);
-   }
-
-   if (!bufObj) {
-      _mesa_error(ctx, GL_INVALID_OPERATION,
-                  "glBindBufferOffsetEXT(invalid buffer=%u)", buffer);
-      return;
+      if (!bufObj) {
+         _mesa_error(ctx, GL_INVALID_OPERATION,
+                     "glBindBufferOffsetEXT(invalid buffer=%u)", buffer);
+         return;
+      }
    }
 
    _mesa_bind_buffer_range_xfb(ctx, obj, index, bufObj, offset, 0);

_______________________________________________
mesa-commit mailing list
mesa-commit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-commit