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

Reply via email to