[Mesa-dev] [PATCH 1/2] st/mesa: move dereference after null check

Erik Faye-Lund kusmabite at gmail.com
Tue Mar 1 22:38:21 UTC 2016


On Tue, Mar 1, 2016 at 10:45 PM, Thomas Hindoe Paaboel Andersen
<phomes at gmail.com> wrote:
> We should not dereference shader before we have done the
> null check.
> ---
>  src/mesa/state_tracker/st_atom_image.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/src/mesa/state_tracker/st_atom_image.c b/src/mesa/state_tracker/st_atom_image.c
> index 4b48bc3..d0f0c42 100644
> --- a/src/mesa/state_tracker/st_atom_image.c
> +++ b/src/mesa/state_tracker/st_atom_image.c
> @@ -48,11 +48,13 @@ st_bind_images(struct st_context *st, struct gl_shader *shader,
>  {
>     unsigned i;
>     struct pipe_image_view images[MAX_IMAGE_UNIFORMS];
> -   struct gl_program_constants *c = &st->ctx->Const.Program[shader->Stage];
> +   struct gl_program_constants *c;
>
>     if (!shader || !st->pipe->set_shader_images)
>        return;
>
> +   c = &st->ctx->Const.Program[shader->Stage];
> +
>     for (i = 0; i < shader->NumImages; i++) {
>        struct gl_image_unit *u = &st->ctx->ImageUnits[shader->ImageUnits[i]];
>        struct st_texture_object *stObj = st_texture_object(u->TexObj);

Looks good, good catch.

Reviewed-by: Erik Faye-Lund <kusmabite at gmail.com>


More information about the mesa-dev mailing list