From 4fd8b3012371a5795a0d272928266c6237e57466 Mon Sep 17 00:00:00 2001 From: Ian Romanick Date: Sun, 21 Dec 2014 12:03:57 -0800 Subject: [PATCH] mesa: Ensure that length is set to zero in _mesa_GetProgramBinary v2: Fix assignment of length. Noticed by Julien Cristau. Signed-off-by: Ian Romanick Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=87516 Reviewed-by: Kenneth Graunke Acked-by: Leight Bade --- src/mesa/main/shaderapi.c | 26 ++++++++++++++++++++------ 1 file changed, 20 insertions(+), 6 deletions(-) diff --git a/src/mesa/main/shaderapi.c b/src/mesa/main/shaderapi.c index 784fa5c16a7..656ee112e55 100644 --- a/src/mesa/main/shaderapi.c +++ b/src/mesa/main/shaderapi.c @@ -1680,16 +1680,35 @@ _mesa_GetProgramBinary(GLuint program, GLsizei bufSize, GLsizei *length, GLenum *binaryFormat, GLvoid *binary) { struct gl_shader_program *shProg; + GLsizei length_dummy; GET_CURRENT_CONTEXT(ctx); shProg = _mesa_lookup_shader_program_err(ctx, program, "glGetProgramBinary"); if (!shProg) return; + /* The ARB_get_program_binary spec says: + * + * "If is NULL, then no length is returned." + * + * Ensure that length always points to valid storage to avoid multiple NULL + * pointer checks below. + */ + if (length != NULL) + length = &length_dummy; + + + /* The ARB_get_program_binary spec says: + * + * "When a program object's LINK_STATUS is FALSE, its program binary + * length is zero, and a call to GetProgramBinary will generate an + * INVALID_OPERATION error. + */ if (!shProg->LinkStatus) { _mesa_error(ctx, GL_INVALID_OPERATION, "glGetProgramBinary(program %u not linked)", shProg->Name); + *length = 0; return; } @@ -1698,12 +1717,7 @@ _mesa_GetProgramBinary(GLuint program, GLsizei bufSize, GLsizei *length, return; } - /* The ARB_get_program_binary spec says: - * - * "If is NULL, then no length is returned." - */ - if (length != NULL) - *length = 0; + *length = 0; (void) binaryFormat; (void) binary; -- 2.30.2