http://bugs.freedesktop.org/show_bug.cgi?id=17659





--- Comment #1 from George Sapountzis <gsapount...@gmail.com>  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.

------------------------------------------------------------------------------
Download Intel&#174; Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev

Reply via email to