From: Brian Paul <brianp@vmware.com>
Date: Fri, 24 Jan 2014 01:18:17 +0000 (-0700)
Subject: st/mesa: add a simple sanity check assertion in st_validate_attachment()
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=e2dd240e326f5af2ad44d78bc245dfcc5c35616d;p=mesa.git

st/mesa: add a simple sanity check assertion in st_validate_attachment()

Reviewed-by: Marek Olšák <marek.olsak@amd.com>
---

diff --git a/src/mesa/state_tracker/st_cb_fbo.c b/src/mesa/state_tracker/st_cb_fbo.c
index eca04b89b5a..780148487f7 100644
--- a/src/mesa/state_tracker/st_cb_fbo.c
+++ b/src/mesa/state_tracker/st_cb_fbo.c
@@ -544,6 +544,12 @@ st_validate_attachment(struct gl_context *ctx,
    gl_format texFormat;
    GLboolean valid;
 
+   /* Sanity check: we must be binding the surface as a (color) render target
+    * or depth/stencil target.
+    */
+   assert(bindings == PIPE_BIND_RENDER_TARGET ||
+          bindings == PIPE_BIND_DEPTH_STENCIL);
+
    /* Only validate texture attachments for now, since
     * st_renderbuffer_alloc_storage makes sure that
     * the format is supported.