[Piglit] [PATCH] shader_runner: Check sscanf return value.

[Brian Paul]

On 11/04/2016 05:30 PM, Vinson Lee wrote:
> Fix Coverity unchecked return value defect.
>
> CID: 1373655
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>   tests/shaders/shader_runner.c | 7 ++++++-
>   1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/tests/shaders/shader_runner.c b/tests/shaders/shader_runner.c
> index 02a6cd7cda82..62fc8120a30c 100644
> --- a/tests/shaders/shader_runner.c
> +++ b/tests/shaders/shader_runner.c
> @@ -2769,8 +2769,13 @@ handle_texparameter(const char *line)
>   		return;
>   	} else if (string_match("border_color ", line)) {
>   		float bc[4];
> +		int count;
>   		line += strlen("border_color ");
> -		sscanf(line, "%f %f %f %f", &bc[0], &bc[1], &bc[2], &bc[3]);
> +		count = sscanf(line, "%f %f %f %f", &bc[0], &bc[1], &bc[2], &bc[3]);
> +		if (count != 4) {
> +			fprintf(stderr, "Could not parse border_color texture parameter.\n");
> +			piglit_report_result(PIGLIT_FAIL);
> +		}
>   		glTexParameterfv(target, GL_TEXTURE_BORDER_COLOR, bc);
>   		return;
>   	} else if (string_match("swizzle_r ", line)) {
>

Reviewed-by: Brian Paul <brianp at vmware.com>