[Piglit] [PATCH v3] shader_runner: make buffer large enough to fit any GL enum

Brian Paul brianp at vmware.com
Mon Mar 6 19:25:19 UTC 2017


On 03/06/2017 10:57 AM, Sandra Koroniewska wrote:
> This fixes a buffer overflow in spec/arb_shader_image_load_store/execution/disable_early_z.shader_test
> ---
>   tests/shaders/shader_runner.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/shaders/shader_runner.c b/tests/shaders/shader_runner.c
> index 89e6e0a5a..b487e8a01 100644
> --- a/tests/shaders/shader_runner.c
> +++ b/tests/shaders/shader_runner.c
> @@ -2821,7 +2821,7 @@ piglit_display(void)
>   		double d[4];
>   		int x, y, z, w, h, l, tex, level;
>   		unsigned ux, uy;
> -		char s[32];
> +		char s[300]; // 300 for safety
>   		enum piglit_result result = PIGLIT_PASS;
>
>   		parse_whitespace(next_line, &line);
>

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

Need me to push this for you?

-Brian



More information about the Piglit mailing list