[Mesa-dev] [Mesa-stable] [PATCH] mesa: return v.value_int64 when the requested type is TYPE_INT64

[Brian Paul]

On 03/13/2014 02:31 AM, Emil Velikov wrote:
> Fixes "Operands don't affect result" defect reported by Coverity.
>
> 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 */
>

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