http://bugs.freedesktop.org/show_bug.cgi?id=17659
--- Comment #1 from George Sapountzis <gsa...@gm...> 2010-03-27 05:49:45 PST ---
(In reply to comment #0)
> Created an attachment (id=19010)
--> (http://bugs.freedesktop.org/attachment.cgi?id=19010) [details]
> half-hearted demo patch which makes valgrind shutup
>
> in src/glx/x11/drisw_glx.c
>
> const __DRIconfig **driver_configs;
>
> psc->swrast->createNewScreen(screen,
> loader_extensions, &driver_configs, psc);
>
> allocates space for the driver_configs which never get released afaics
>
You are right that they are not freed. The problem exists for actual DRI
drivers also and for other loaders (xserver/glx ...).
So unless the configs are used by the loader after screen destruction time, I
think that they should be freed by the common dri code under
src/mesa/drivers/dri/common in the .DestroyScreen hook.
--
Configure bugmail: http://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
|
