diff options
author | Lionel Landwerlin <lionel.g.landwerlin@intel.com> | 2019-11-01 19:43:00 +0200 |
---|---|---|
committer | Dylan Baker <dylan@pnwbakers.com> | 2019-11-05 09:17:38 -0800 |
commit | 6daaf66f662ed5d3236c654721910506cc7f4c93 (patch) | |
tree | 8722844b41641c81f78ad3a7db4a5dea1a20165d | |
parent | 4d21f802b56af984f2f2bfa9e40b84da550662b7 (diff) |
mesa: check draw buffer completeness on glClearBufferfi/glClearBufferiv
Signed-off-by: Lionel Landwerlin <lionel.g.landwerlin@intel.com>
Cc: <mesa-stable@lists.freedesktop.org>
Reviewed-by: Marek Olšák <marek.olsak@amd.com>
(cherry picked from commit 88d665830f27087cb2188e03b0b734acc144c593)
-rw-r--r-- | src/mesa/main/clear.c | 12 |
1 files changed, 12 insertions, 0 deletions
diff --git a/src/mesa/main/clear.c b/src/mesa/main/clear.c index 9221c7287a3..c5d612f4a6c 100644 --- a/src/mesa/main/clear.c +++ b/src/mesa/main/clear.c @@ -350,6 +350,12 @@ clear_bufferiv(struct gl_context *ctx, GLenum buffer, GLint drawbuffer, _mesa_update_state( ctx ); } + if (!no_error && ctx->DrawBuffer->_Status != GL_FRAMEBUFFER_COMPLETE_EXT) { + _mesa_error(ctx, GL_INVALID_FRAMEBUFFER_OPERATION_EXT, + "glClearBufferiv(incomplete framebuffer)"); + return; + } + switch (buffer) { case GL_STENCIL: /* Page 264 (page 280 of the PDF) of the OpenGL 3.0 spec says: @@ -686,6 +692,12 @@ clear_bufferfi(struct gl_context *ctx, GLenum buffer, GLint drawbuffer, drawbuffer); return; } + + if (ctx->DrawBuffer->_Status != GL_FRAMEBUFFER_COMPLETE_EXT) { + _mesa_error(ctx, GL_INVALID_FRAMEBUFFER_OPERATION_EXT, + "glClearBufferfi(incomplete framebuffer)"); + return; + } } if (ctx->RasterDiscard) |