[MPlayer-dev-eng] memory leak in the libaf/af.c

Uoti Urpala uoti.urpala at pp1.inet.fi
Sat Feb 10 01:34:35 CET 2007


On Sat, 2007-02-10 at 03:27 +0300, Andrew Savchenko wrote:
> In function af_data_t* af_play() (libaf/af.c, line 507)
> af pointer is initialized (af_instance_t* af=s->first;) but it is 
> not freed anywhere. This memory leak is confirmed with valgrind 
> and occurs when seeking to another file in playlist.
> 
> Obvious patch seems to fix this. Please review.

This "obvious" patch looks obviously wrong. The pointer is initialized
to point to the existing filter chain; it shouldn't be freed after
filtering one chunk of data.




More information about the MPlayer-dev-eng mailing list