[PULL] Xorg/darwin related build fixes, Xquartz crash fix

Jeremy Huddleston jeremyhu at apple.com
Mon Jan 16 14:35:12 PST 2012


On Jan 16, 2012, at 12:09 PM, Keith Packard wrote:

> On Sun, 15 Jan 2012 02:35:03 -0800, Jeremy Huddleston <jeremyhu at apple.com> wrote:
> 
>> Jeremy Huddleston (5):
>>      Revert "glx: don't leak fbconfigs"
> 
> The comment on this one says that the correct fix is in
> __glXDRIscreenDestroy, but I don't see those functions doing this. Do we
> need another patch that adds the fbconfigs deallocation there?


Sorry if the comment was ambiguous, but I did not mean to imply that the fix was present already or provided.  Reverting this patch does return the memory leak when using glxdri, and a followup needs to be provided to fix it properly in __glXDRIscreenDestroy.  I do not use glxdri, so I wanted to leave that to someone else (Paulo) to implement.

--Jeremy


More information about the xorg-devel mailing list