On Fri, Nov 7, 2014 at 6:05 PM, Luca Barbato <[email protected]> wrote: > On 04/11/14 15:36, Vittorio Giovara wrote: >> >> From: Michael Niedermayer <[email protected]> >> >> CC: [email protected] >> Bug-Id: CID 991852 >> --- >> libavcodec/hpeldsp.h | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/libavcodec/hpeldsp.h b/libavcodec/hpeldsp.h >> index 7ffed1a..d037cba 100644 >> --- a/libavcodec/hpeldsp.h >> +++ b/libavcodec/hpeldsp.h >> @@ -69,7 +69,7 @@ typedef struct HpelDSPContext { >> >> /** >> * Halfpel motion compensation with no rounding (a+b)>>1. >> - * this is an array[2][4] of motion compensation functions for 2 >> + * this is an array[4][4] of motion compensation functions for 2 >> * horizontal blocksizes (8,16) and the 4 halfpel positions<br> >> * *pixels_tab[ 0->16xH 1->8xH ][ xhalfpel + 2*yhalfpel ] >> * @param block destination where the result is stored >> @@ -77,7 +77,7 @@ typedef struct HpelDSPContext { >> * @param line_size number of bytes in a horizontal line of block >> * @param h height >> */ >> - op_pixels_func put_no_rnd_pixels_tab[2][4]; >> + op_pixels_func put_no_rnd_pixels_tab[4][4]; >> >> /** >> * Halfpel motion compensation with no rounding (a+b)>>1. >> > > Ok if you checked that 4 is ok.
ff_init_me() explicitly accesses elements past the second one. -- Vittorio _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
