[Piglit] [PATCH] arb_tessellation_shader: Check piglit_link_check_status results.

Marek Olšák maraeo at gmail.com
Mon Nov 30 01:47:45 PST 2015


Reviewed-by. Marek Olšák <marek.olsak at amd.com>

Marek

On Mon, Nov 30, 2015 at 8:46 AM, Vinson Lee <vlee at freedesktop.org> wrote:
> Fixes Coverity "unchecked return value" defects.
>
> Signed-off-by: Vinson Lee <vlee at freedesktop.org>
> ---
>  tests/spec/arb_tessellation_shader/invalid-get-program-params.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/tests/spec/arb_tessellation_shader/invalid-get-program-params.c b/tests/spec/arb_tessellation_shader/invalid-get-program-params.c
> index 4cbe29a..c3347d8 100644
> --- a/tests/spec/arb_tessellation_shader/invalid-get-program-params.c
> +++ b/tests/spec/arb_tessellation_shader/invalid-get-program-params.c
> @@ -78,11 +78,11 @@ piglit_init(int argc, char **argv)
>
>         tcs_prog = glCreateShaderProgramv(GL_TESS_CONTROL_SHADER, 1,
>                                           (const GLchar *const*)&tcs_source);
> -       piglit_link_check_status(tcs_prog);
> +       pass = piglit_link_check_status(tcs_prog) && pass;
>
>         tes_prog = glCreateShaderProgramv(GL_TESS_EVALUATION_SHADER, 1,
>                                           (const GLchar *const*)&tes_source);
> -       piglit_link_check_status(tes_prog);
> +       pass = piglit_link_check_status(tes_prog) && pass;
>
>         for (i = 0; i < ARRAY_SIZE(tes_params); ++i ) {
>                 pass = piglit_check_gl_error(GL_NO_ERROR) && pass;
> --
> 2.6.3
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit


More information about the Piglit mailing list