[Mesa-dev] [PATCH] llvmpipe: Silence Coverity incorrect sizeof expression defect.

Jose Fonseca jfonseca at vmware.com
Mon Aug 13 04:03:31 PDT 2012


Good catch. Thanks.

Reviewed-by: Jose Fonseca <jfonseca at vmware.com>


----- Original Message -----
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  src/gallium/drivers/llvmpipe/lp_state_setup.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/gallium/drivers/llvmpipe/lp_state_setup.c
> b/src/gallium/drivers/llvmpipe/lp_state_setup.c
> index 1d5e50b..469a459 100644
> --- a/src/gallium/drivers/llvmpipe/lp_state_setup.c
> +++ b/src/gallium/drivers/llvmpipe/lp_state_setup.c
> @@ -585,7 +585,7 @@ set_noalias(LLVMBuilderRef builder,
>  	    int nr_args)
>  {
>     int i;
> -   for(i = 0; i < Elements(arg_types); ++i)
> +   for(i = 0; i < nr_args; ++i)
>        if(LLVMGetTypeKind(arg_types[i]) == LLVMPointerTypeKind)
>           LLVMAddAttribute(LLVMGetParam(function, i),
>  			  LLVMNoAliasAttribute);
> --
> 1.7.11.1
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev
> 


More information about the mesa-dev mailing list