Reviewed-by: Ilia Mirkin <imir...@alum.mit.edu>
On Mon, Dec 14, 2015 at 12:07 PM, Samuel Pitoiset <samuel.pitoi...@gmail.com> wrote: > Spotted by Coverity. > > Signed-off-by: Samuel Pitoiset <samuel.pitoi...@gmail.com> > --- > src/gallium/drivers/nouveau/nvc0/nvc0_shader_state.c | 5 +---- > 1 file changed, 1 insertion(+), 4 deletions(-) > > diff --git a/src/gallium/drivers/nouveau/nvc0/nvc0_shader_state.c > b/src/gallium/drivers/nouveau/nvc0/nvc0_shader_state.c > index 7e2e999..5e84ca9 100644 > --- a/src/gallium/drivers/nouveau/nvc0/nvc0_shader_state.c > +++ b/src/gallium/drivers/nouveau/nvc0/nvc0_shader_state.c > @@ -236,11 +236,8 @@ nvc0_gmtyprog_validate(struct nvc0_context *nvc0) > struct nouveau_pushbuf *push = nvc0->base.pushbuf; > struct nvc0_program *gp = nvc0->gmtyprog; > > - if (gp) > - nvc0_program_validate(nvc0, gp); > - > /* we allow GPs with no code for specifying stream output state only */ > - if (gp && gp->code_size) { > + if (gp && nvc0_program_validate(nvc0, gp) && gp->code_size) { > const bool gp_selects_layer = !!(gp->hdr[13] & (1 << 9)); > > BEGIN_NVC0(push, NVC0_3D(MACRO_GP_SELECT), 1); > -- > 2.6.4 > > _______________________________________________ > mesa-dev mailing list > mesa-dev@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/mesa-dev _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev