Adam Jackson <[email protected]> writes:

> Signed-off-by: Adam Jackson <[email protected]>
> ---
>  hw/dmx/glxProxy/glxcmds.c | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/hw/dmx/glxProxy/glxcmds.c b/hw/dmx/glxProxy/glxcmds.c
> index 964db40..f097906 100644
> --- a/hw/dmx/glxProxy/glxcmds.c
> +++ b/hw/dmx/glxProxy/glxcmds.c
> @@ -2017,6 +2017,7 @@ CreateGLXPixmap(__GLXclientState * cl,
>      }
>  
>      if (!(AddResource(glxpixmapId, __glXPixmapRes, pGlxPixmap))) {
> +        free(pGlxPixmap->be_xids);
>          free(pGlxPixmap);
>          return BadAlloc;
>      }

I think you missed a return path about 8 lines before this:

        else {
            client->errorValue = (visual ? visual : fbconfigId);
            free(pGlxPixmap);
            return BadValue;
        }

        XFlush(dpy);
    }

    if (!(AddResource(glxpixmapId, __glXPixmapRes, pGlxPixmap))) {
        free(pGlxPixmap);
        return BadAlloc;
    }


-- 
[email protected]

Attachment: pgpgWoEj1xjrr.pgp
Description: PGP signature

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to