Hi,

On Thu, 2009-05-07 at 15:56 +0200, Denis Oliver Kropp wrote:

> > I think I already reported this here quite a while ago, but the change
> > was never committed and still sits in my local tree. Please review and
> > tell me if it is OK to push this change to the git repository.
> 
> You should at least make sure that multiple interface implementations
> in one shared module or linked statically into the application binary
> are still working. There was a double free issue when one dynamic lib
> (un)registered multiple interfaces.

I didn't check that and I am not sure how to check that. However
valgrind showed a memory leak and if you look at the code you will see
that impl is freed a few lines below. Shouldn't we also free the
filename field of impl then?


Sven


_______________________________________________
directfb-dev mailing list
directfb-dev@directfb.org
http://mail.directfb.org/cgi-bin/mailman/listinfo/directfb-dev

Reply via email to