[vlc-devel] Re: A possible memory leak in VLC src/misc/mtime.c

[Gildas Bazin]

On Monday 10 January 2005 13:02, stage.nexvision at laposte.net wrote:
> Hi all,
> 
>    I was just looking into your code, and I see that the mdate function in 
mtime.c is 
> initializing a CriticalSection object but never delete it. Of course, it 
is easier to do it 
> this way (initialized once because it's static) but there is no way to 
delete the critical 
> section.
> 
> That CS could be embedded in a global object, because it is unique in the 
whole 
> process, initialized in the global init code, and free in the global free 
code. 
> 

In this case it would only makes sense to delete the critical section when 
the process is terminating... which renders this delete quite useless.

--
Gildas

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html