[Piglit] [PATCH] arb_query_buffer_object: Fix format-security warning.

2016-05-10 Thread Vinson Lee
qbo.c: In function ‘run_subtest_and_present’:
qbo.c:269:2: warning: format not a string literal and no format arguments 
[-Wformat-security]
  piglit_report_subtest_result(r, subtest_name);
  ^

Signed-off-by: Vinson Lee 
---
 tests/spec/arb_query_buffer_object/qbo.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/spec/arb_query_buffer_object/qbo.c 
b/tests/spec/arb_query_buffer_object/qbo.c
index bd47dd2a3989..1bcbe1035fad 100644
--- a/tests/spec/arb_query_buffer_object/qbo.c
+++ b/tests/spec/arb_query_buffer_object/qbo.c
@@ -266,7 +266,7 @@ run_subtest_and_present(void)
asprintf(&subtest_name, "query-%s-%s",
 piglit_get_gl_enum_name(query_type),
 sync_mode_names[sync_mode]);
-   piglit_report_subtest_result(r, subtest_name);
+   piglit_report_subtest_result(r, "%s", subtest_name);
free(subtest_name);
return r;
 }
-- 
2.7.4

___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit


Re: [Piglit] [PATCH] arb_query_buffer_object: Fix format-security warning.

2016-05-10 Thread Dylan Baker
Quoting Vinson Lee (2016-05-10 17:48:07)
> qbo.c: In function ‘run_subtest_and_present’:
> qbo.c:269:2: warning: format not a string literal and no format arguments 
> [-Wformat-security]
>   piglit_report_subtest_result(r, subtest_name);
>   ^
> 
> Signed-off-by: Vinson Lee 
> ---
>  tests/spec/arb_query_buffer_object/qbo.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/spec/arb_query_buffer_object/qbo.c 
> b/tests/spec/arb_query_buffer_object/qbo.c
> index bd47dd2a3989..1bcbe1035fad 100644
> --- a/tests/spec/arb_query_buffer_object/qbo.c
> +++ b/tests/spec/arb_query_buffer_object/qbo.c
> @@ -266,7 +266,7 @@ run_subtest_and_present(void)
> asprintf(&subtest_name, "query-%s-%s",
>  piglit_get_gl_enum_name(query_type),
>  sync_mode_names[sync_mode]);
> -   piglit_report_subtest_result(r, subtest_name);
> +   piglit_report_subtest_result(r, "%s", subtest_name);
> free(subtest_name);
> return r;
>  }
> -- 
> 2.7.4
> 
> ___
> Piglit mailing list
> Piglit@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/piglit

Reviewed-by: Dylan Baker 


signature.asc
Description: signature
___
Piglit mailing list
Piglit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/piglit