Re: [Libvisual-devel] Libvisual mem bug

[Dennis S. <sy...@yo...>]

On Tue, 2004-09-28 at 00:59 +0300, Vitaly V. Bursov wrote:
> The problem is that you're trying to free() member of a struct.
> ##I.e. There is
> 
> struct _VisParamContainer {
>         VisList          entries;
>         VisEventQueue   *eventqueue;
> };
> 
> ##And
>         visual_list_destroy (&paramcontainer->entries, param_list_destroy);
> ##And
> 
> int visual_list_destroy (VisList *list, visual_list_destroy_func_t destroyer)   
> {
> 	....
>         visual_list_free (list);
> 
>         return 0;
> }
> 
> 
> That is it.
> 
> I think stuff like
>         VisList          entries;
> should be changed to
>         VisList          *entries;
> 

Wow, thank you for tracking it down, believe it or not but I was
looking at that while doing the xmms plugin as well, but didn't
fixed it yet. I'm comitting a fix to CVS in a minute.

Thanks,
Dennis