Re: [FFmpeg-devel] [PATCH] libavfilter: avoid UB nullptr-with-offset.

2021-02-16 Thread Jeremy Leconte
You're right, it's just a case of NULL + 0. The problem is that it gets caught by clang -fsanitize=undefined. On Tue, Feb 16, 2021 at 4:59 PM Andreas Rheinhardt < andreas.rheinha...@gmail.com> wrote: > Jeremy Leconte: > > --- > > libavfilter/vf_scale.c | 2 +- &

Re: [FFmpeg-devel] [PATCH] libavfilter: avoid UB nullptr-with-offset.

2021-02-16 Thread Jeremy Leconte
Andreas Rheinhardt: > Is this just a case of NULL + 0 or is the offset != 0? > > - Andreas > You're right, it's just a case of NULL + 0. My issue is that it gets caught by clang -fsanitize=undefined. Jeremy ___ ffmpeg-devel mailing list ffmpeg-devel@ffmp

[FFmpeg-devel] [PATCH] libavfilter: avoid UB nullptr-with-offset.

2021-02-16 Thread Jeremy Leconte
--- libavfilter/vf_scale.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavfilter/vf_scale.c b/libavfilter/vf_scale.c index 58eee96744..98cef5eb4b 100644 --- a/libavfilter/vf_scale.c +++ b/libavfilter/vf_scale.c @@ -631,7 +631,7 @@ static int scale_slice(AVFilterLink *link

[FFmpeg-devel] [PATCH] libswscale: avoid UB nullptr-with-offset.

2020-12-23 Thread Jeremy Leconte
Great news that the patch is accepted. Just updated the author is this version. Thanks! --- libswscale/slice.c| 12 libswscale/swscale_unscaled.c | 4 +--- 2 files changed, 5 insertions(+), 11 deletions(-) diff --git a/libswscale/slice.c b/libswscale/slice.c index 7849b7

Re: [FFmpeg-devel] [PATCH] libswscale: avoid UB nullptr-with-offset.

2020-12-23 Thread Jeremy Leconte
On Wed, Dec 23, 2020 at 10:14 PM Michael Niedermayer wrote: > patch should be ok, is the author intended to be like this: > Author: jleconte > I mean theres no full name in there > That's great news! If it's ok on your side, I'm good with the author being 'jleconte'. ___