[Mesa-dev] [PATCH v3] i965: Prevent coverity warning

Iago Toral itoral at igalia.com
Wed Feb 1 07:27:24 UTC 2017


Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>

On Tue, 2017-01-31 at 15:20 -0500, Robert Foss wrote:
> Add assert checking that num_sources is never larger than 3.
> 
> This prevents Coverity from concluding that the unhandled
> cases of num_sources not being 0-3 are relevant.
> 
> Coverity-Id: 1399480-1399489
> Signed-off-by: Robert Foss <robert.foss at collabora.com>
> Reviewed-by: Anuj Phogat <anuj.phogat at gmail.com>
> ---
>  src/mesa/drivers/dri/i965/brw_eu_validate.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/mesa/drivers/dri/i965/brw_eu_validate.c
> b/src/mesa/drivers/dri/i965/brw_eu_validate.c
> index 226539445d..64615af44a 100644
> --- a/src/mesa/drivers/dri/i965/brw_eu_validate.c
> +++ b/src/mesa/drivers/dri/i965/brw_eu_validate.c
> @@ -177,6 +177,7 @@ num_sources_from_inst(const struct
> gen_device_info *devinfo,
>           return 0;
>        }
>     } else {
> +      assert(desc->nsrc < 4);
>        return desc->nsrc;
>     }
>  


More information about the mesa-dev mailing list