Reviewed-by: Ilia Mirkin <imir...@alum.mit.edu>
On Sat, Feb 18, 2017 at 12:51 PM, Pierre Moreau <pierre.mor...@free.fr> wrote: > Function arguments do not have an "origin" instruction, causing a > NULL-pointer dereference without this check. > > Signed-off-by: Pierre Moreau <pierre.mor...@free.fr> > --- > src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp > b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp > index 79403c93df..d358abc5bd 100644 > --- a/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp > +++ b/src/gallium/drivers/nouveau/codegen/nv50_ir_peephole.cpp > @@ -410,6 +410,8 @@ ConstantFolding::findOriginForTestWithZero(Value *value) > if (!value) > return NULL; > Instruction *insn = value->getInsn(); > + if (!insn) > + return NULL; > > if (insn->asCmp() && insn->op != OP_SLCT) > return insn->asCmp(); > -- > 2.11.1 > _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev