[Mesa-dev] [PATCH 8/9] Replace another malloc/memset-0 combination with calloc

[Brian Paul]

On 09/05/2012 12:42 AM, Matt Turner wrote:
> ---
>   src/glx/indirect_glx.c |    3 +--
>   1 files changed, 1 insertions(+), 2 deletions(-)
>
> diff --git a/src/glx/indirect_glx.c b/src/glx/indirect_glx.c
> index f3cc1e7..ff869e2 100644
> --- a/src/glx/indirect_glx.c
> +++ b/src/glx/indirect_glx.c
> @@ -361,14 +361,13 @@ indirect_create_context(struct glx_screen *psc,
>      glx_context_init(gc, psc, mode);
>      gc->isDirect = GL_FALSE;
>      gc->vtable =&indirect_context_vtable;
> -   state = malloc(sizeof(struct __GLXattributeRec));
> +   state = calloc(1, sizeof(struct __GLXattributeRec));
>      if (state == NULL) {
>         /* Out of memory */
>         free(gc);
>         return NULL;
>      }
>      gc->client_state_private = state;
> -   memset(gc->client_state_private, 0, sizeof(struct __GLXattributeRec));
>      state->NoDrawArraysProtocol = (getenv("LIBGL_NO_DRAWARRAYS") != NULL);
>
>      /*

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