Re: [FFmpeg-devel] [PATCH] avutil/mem: use GCC builtins to check for overflow in av_size_mult()

2021-05-31 Thread Carl Eugen Hoyos
Am Do., 27. Mai 2021 um 17:08 Uhr schrieb James Almer : > > Signed-off-by: James Almer > --- > libavutil/mem.h | 9 - > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/libavutil/mem.h b/libavutil/mem.h > index e21a1feaae..c876111afb 100644 > --- a/libavutil/mem.h > +++

Re: [FFmpeg-devel] [PATCH] avutil/mem: use GCC builtins to check for overflow in av_size_mult()

2021-05-31 Thread Nicolas George
Anton Khirnov (12021-05-31): > This is starting to look way too ugly for something exposed in a public > header. The fact it resides in a public header is just a limitation of C's linking model. The code itself is internal source, users should not look at it. Regards, -- Nicolas George

Re: [FFmpeg-devel] [PATCH] avutil/mem: use GCC builtins to check for overflow in av_size_mult()

2021-05-31 Thread Anton Khirnov
Quoting James Almer (2021-05-27 17:06:57) > Signed-off-by: James Almer > --- > libavutil/mem.h | 9 - > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/libavutil/mem.h b/libavutil/mem.h > index e21a1feaae..c876111afb 100644 > --- a/libavutil/mem.h > +++ b/libavutil/mem.h

Re: [FFmpeg-devel] [PATCH] avutil/mem: use GCC builtins to check for overflow in av_size_mult()

2021-05-30 Thread James Almer
On 5/27/2021 12:06 PM, James Almer wrote: Signed-off-by: James Almer --- libavutil/mem.h | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavutil/mem.h b/libavutil/mem.h index e21a1feaae..c876111afb 100644 --- a/libavutil/mem.h +++ b/libavutil/mem.h @@ -674,11

[FFmpeg-devel] [PATCH] avutil/mem: use GCC builtins to check for overflow in av_size_mult()

2021-05-27 Thread James Almer
Signed-off-by: James Almer --- libavutil/mem.h | 9 - 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/libavutil/mem.h b/libavutil/mem.h index e21a1feaae..c876111afb 100644 --- a/libavutil/mem.h +++ b/libavutil/mem.h @@ -674,11 +674,18 @@ void *av_dynarray2_add(void