[SDL] SDL Mixer major bug

[Bruno Coudoin]

Le lundi 27 novembre 2006 à 23:10 +0100, Patrice Mandin a écrit :
> Le Mon, 27 Nov 2006 22:47:33 +0100
> Bruno Coudoin <bruno.coudoin at free.fr> a écrit:
> 
> > I provided a patch but if I get no feedback from the authors, what can
> > I do. I am sad to see that many distro have the issue, it make
> > GCompris and probably other games unstable (at least, Mandriva 2007
> > has the issue).
> 
> Hello (again),
> 
> I just looked your patch, and this is what I was fearing: I am the
> cause. From your patch, it seems the timidity path list is filled only
> the first time its init function is called (because of the static
> variable I added). So the second time the timidity function is called,
> the path list is kept empty with NULL string pointer, hence the crash.
> 
You don't need your static variable anymore. With my patch, pathlist is
inited and freed at each open/close. The "pathlist = NULL" in
free_pathlist() does the job to make sure we reinit it at next open.

> I'll try to fix this as soon as possible.

Excellent new, thank a lot from the GCompris project.

-- 
Bruno Coudoin
http://gcompris.net Free educational software for kids
http://ofset.org    Free educational software for all
http://toulibre.org Logiciel Libre à Toulouse