On Jan 16, 2012, at 12:09 PM, Keith Packard wrote: > On Sun, 15 Jan 2012 02:35:03 -0800, Jeremy Huddleston <jerem...@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 _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel