[PULL] Xorg/darwin related build fixes, Xquartz crash fix
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.
More information about the xorg-devel