Re: [FFmpeg-devel] [PATCH 2/2] avcodec/cfhd: Use bytestream2 for peaks

2018-07-18 Thread Michael Niedermayer
On Sun, Jul 15, 2018 at 08:51:36PM +0100, Kieran Kunhya wrote: > On Sat, 14 Jul 2018 at 00:14 Michael Niedermayer > wrote: > > > This fixes out of array accesses > > No testcase known > > > > Signed-off-by: Michael Niedermayer > > > > Looks ok if Gagandeep is ok noone seems to have objected so

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/cfhd: Use bytestream2 for peaks

2018-07-15 Thread Kieran Kunhya
On Sat, 14 Jul 2018 at 00:14 Michael Niedermayer wrote: > This fixes out of array accesses > No testcase known > > Signed-off-by: Michael Niedermayer > Looks ok if Gagandeep is ok Kieran ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ff

[FFmpeg-devel] [PATCH 2/2] avcodec/cfhd: Use bytestream2 for peaks

2018-07-13 Thread Michael Niedermayer
This fixes out of array accesses No testcase known Signed-off-by: Michael Niedermayer --- libavcodec/cfhd.c | 10 +- libavcodec/cfhd.h | 3 ++- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/libavcodec/cfhd.c b/libavcodec/cfhd.c index ef5ebe42c5..76838f0869 100644 --- a/