This fixes: GL45-CTS.direct_state_access.queries_errors The ARB_direct_state_access spec agrees.
Signed-off-by: Dave Airlie <airl...@redhat.com> --- src/mesa/main/queryobj.c | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/mesa/main/queryobj.c b/src/mesa/main/queryobj.c index 7a70b59..71bacc4 100644 --- a/src/mesa/main/queryobj.c +++ b/src/mesa/main/queryobj.c @@ -754,6 +754,12 @@ get_query_object(struct gl_context *ctx, const char *func, return; } + if (offset < 0) { + _mesa_error(ctx, GL_INVALID_VALUE, + "%s(offset is negative)", func); + return; + } + if (buf && buf != ctx->Shared->NullBufferObj) { bool is_64bit = ptype == GL_INT64_ARB || ptype == GL_UNSIGNED_INT64_ARB; -- 2.7.4 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev