[Piglit] [PATCH] cl: Initialize errNo on invalid image type.

[Serge Martin]

On Friday 01 July 2016 21:18:15 Vinson Lee wrote:
> Fixes sometimes-uninitialized warning.
> 
> piglit-util-cl.c:1052:13: warning: variable 'errNo' is used uninitialized
> whenever 'if' condition is false [-Wsometimes-uninitialized] } else if
> (desc->image_type == CL_MEM_OBJECT_IMAGE3D) {
>                    ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> piglit-util-cl.c:1062:28: note: uninitialized use occurs here
>         if(!piglit_cl_check_error(errNo, CL_SUCCESS)) {
>                                   ^~~~~
> piglit-util-cl.c:1052:9: note: remove the 'if' if its condition is always
> true } else if (desc->image_type == CL_MEM_OBJECT_IMAGE3D) {
>                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> piglit-util-cl.c:1040:14: note: initialize the variable 'errNo' to silence
> this warning cl_int errNo;
>                     ^
>                      = 0
> 
> Fixes: 910cd6c222930 ("cl: add image and sampler utility functions")
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  tests/util/piglit-util-cl.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/util/piglit-util-cl.c b/tests/util/piglit-util-cl.c
> index 7e7d9850be5c..b5b7fce550f6 100644
> --- a/tests/util/piglit-util-cl.c
> +++ b/tests/util/piglit-util-cl.c
> @@ -1055,6 +1055,7 @@ piglit_cl_create_image(piglit_cl_context context,
> cl_mem_flags flags, desc->image_depth, 0, 0,
>  		                        NULL, &errNo);
>  	} else {
> +		errNo = CL_INVALID_MEM_OBJECT;

This not an error returned by clCreateImage*D. If your ok with that I would 
prefer to push it with CL_INVALID_OPERATION.

>  		fprintf(stderr,
>  		        "Invalid image mem object type: %s\n",
>  		        piglit_cl_get_enum_name(desc->image_type));