[Mesa-stable] [PATCH] mesa: return v.value_int64 when the requested type is TYPE_INT64
Ian Romanick
idr at freedesktop.org
Thu Mar 13 08:31:19 PDT 2014
On 03/13/2014 01:31 AM, Emil Velikov wrote:
> Fixes "Operands don't affect result" defect reported by Coverity.
D'oh. So... we obviously don't have any piglit tests that hit these
cases. Would you mind whipping up something simple?
> Cc: "9.2 10.0 10.1" <mesa-stable at lists.freedesktop.org>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
> src/mesa/main/get.c | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/src/mesa/main/get.c b/src/mesa/main/get.c
> index b190851..88cf202 100644
> --- a/src/mesa/main/get.c
> +++ b/src/mesa/main/get.c
> @@ -1997,7 +1997,7 @@ _mesa_GetBooleani_v( GLenum pname, GLuint index, GLboolean *params )
> params[3] = INT_TO_BOOLEAN(v.value_int_4[3]);
> break;
> case TYPE_INT64:
> - params[0] = INT64_TO_BOOLEAN(v.value_int);
> + params[0] = INT64_TO_BOOLEAN(v.value_int64);
> break;
> default:
> ; /* nothing - GL error was recorded */
> @@ -2042,7 +2042,7 @@ _mesa_GetIntegeri_v( GLenum pname, GLuint index, GLint *params )
> params[3] = v.value_int_4[3];
> break;
> case TYPE_INT64:
> - params[0] = INT64_TO_INT(v.value_int);
> + params[0] = INT64_TO_INT(v.value_int64);
> break;
> default:
> ; /* nothing - GL error was recorded */
> @@ -2067,7 +2067,7 @@ _mesa_GetInteger64i_v( GLenum pname, GLuint index, GLint64 *params )
> params[3] = v.value_int_4[3];
> break;
> case TYPE_INT64:
> - params[0] = v.value_int;
> + params[0] = v.value_int64;
> break;
> default:
> ; /* nothing - GL error was recorded */
>
More information about the mesa-stable
mailing list