[Ffmpeg-devel] [PATCH] register_protocol

[Michael Niedermayer]

Hi

On Mon, Jan 30, 2006 at 10:40:48AM +0100, Michel Bardiaux wrote:
> Michael Niedermayer wrote:
> >Hi
> >
> >On Tue, Jan 03, 2006 at 12:00:26PM +0100, Michel Bardiaux wrote:
> >
> >>The current implementation has several problems:
> >>
> >>(1) Call av_register_all, the register one of yours, then
> >>av_register_all again: yours is lost... Worse: register yours, call
> >>av_register_all, register yours again, you get... only yours.
> >
> >
> >if(p == protocol)
> >    return -1;
> >
> >
> >
> >>(2) Relies on a linked list that actually resides in klientland.
> >
> >
> >and yours is bloated and leaks :)
> 
> Could you point out the leak?

av_malloc() ... what else? load lib, register something, unload ...
it should be av_malloc_static()

[...]
-- 
Michael