Re: [FFmpeg-devel] [PATCH] avutil/mem: set default allocation limit to "unlimited"

2015-03-31 Thread wm4
On Tue, 31 Mar 2015 15:53:36 + (UTC) Carl Eugen Hoyos wrote: > Michael Niedermayer gmx.at> writes: > > > -static size_t max_alloc_size= INT_MAX; > > +static size_t max_alloc_size= SIZE_MAX; > > What would this fix? Nothing yet, because both AVPacket and AVFrame still use ints for size qua

Re: [FFmpeg-devel] [PATCH] avutil/mem: set default allocation limit to "unlimited"

2015-03-31 Thread Carl Eugen Hoyos
Michael Niedermayer gmx.at> writes: > -static size_t max_alloc_size= INT_MAX; > +static size_t max_alloc_size= SIZE_MAX; What would this fix? I remember several reports about FFmpeg "leaking" memory where the users just meant that one of the libraries allocated more memory than the user mean

Re: [FFmpeg-devel] [PATCH] avutil/mem: set default allocation limit to "unlimited"

2015-03-31 Thread wm4
On Tue, 31 Mar 2015 16:55:00 +0200 Michael Niedermayer wrote: > Signed-off-by: Michael Niedermayer > --- > libavutil/mem.c |2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavutil/mem.c b/libavutil/mem.c > index 5fd2395..2849188 100644 > --- a/libavutil/mem.c > +++

[FFmpeg-devel] [PATCH] avutil/mem: set default allocation limit to "unlimited"

2015-03-31 Thread Michael Niedermayer
Signed-off-by: Michael Niedermayer --- libavutil/mem.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/mem.c b/libavutil/mem.c index 5fd2395..2849188 100644 --- a/libavutil/mem.c +++ b/libavutil/mem.c @@ -66,7 +66,7 @@ void free(void *ptr); * dynamic libraries