On 27 November 2013 17:11, Vittorio Giovara <[email protected]> wrote:
> From: Michael Niedermayer <[email protected]>
>
> Fixes out of array accesses.
>
> Found_by: ami_stuff
> ---
>  libavcodec/pngdsp.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/pngdsp.c b/libavcodec/pngdsp.c
> index afd9093..5cc41c4 100644
> --- a/libavcodec/pngdsp.c
> +++ b/libavcodec/pngdsp.c
> @@ -31,7 +31,7 @@
>  static void add_bytes_l2_c(uint8_t *dst, uint8_t *src1, uint8_t *src2, int w)
>  {
>      long i;
> -    for (i = 0; i <= w - sizeof(long); i += sizeof(long)) {
> +    for (i = 0; i <= w - (int)sizeof(long); i += sizeof(long)) {
>          long a = *(long *)(src1 + i);
>          long b = *(long *)(src2 + i);
>          *(long *)(dst + i) = ((a & pb_7f) + (b & pb_7f)) ^ ((a ^ b) & pb_80);
> --

looks OK except for using longs in the first place
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to