<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Mar 2, 2016 at 12:33 AM, Brian Paul <span dir="ltr"><<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 03/01/2016 02:45 PM, Thomas Hindoe Paaboel Andersen wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
We should not dereference shader before we have done the<br>
null check.<br>
---<br>
  src/mesa/state_tracker/st_atom_image.c | 4 +++-<br>
  1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/src/mesa/state_tracker/st_atom_image.c b/src/mesa/state_tracker/st_atom_image.c<br>
index 4b48bc3..d0f0c42 100644<br>
--- a/src/mesa/state_tracker/st_atom_image.c<br>
+++ b/src/mesa/state_tracker/st_atom_image.c<br>
@@ -48,11 +48,13 @@ st_bind_images(struct st_context *st, struct gl_shader *shader,<br>
  {<br>
     unsigned i;<br>
     struct pipe_image_view images[MAX_IMAGE_UNIFORMS];<br>
-   struct gl_program_constants *c = &st->ctx->Const.Program[shader->Stage];<br>
+   struct gl_program_constants *c;<br>
<br>
     if (!shader || !st->pipe->set_shader_images)<br>
        return;<br>
<br>
+   c = &st->ctx->Const.Program[shader->Stage];<br>
+<br>
     for (i = 0; i < shader->NumImages; i++) {<br>
        struct gl_image_unit *u = &st->ctx->ImageUnits[shader->ImageUnits[i]];<br>
        struct st_texture_object *stObj = st_texture_object(u->TexObj);<br>
<br>
</blockquote>
<br></span>
For both:<br>
Reviewed-by: Brian Paul <<a href="mailto:brianp@vmware.com" target="_blank">brianp@vmware.com</a>><br>
<br>
Do you need someone to push these for you?<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>Yes, please push. Thanks! <br></div></div><br></div></div>