On 13/03/14 16:27, Vittorio Giovara wrote:
> From: Michael Niedermayer <michae...@gmx.at>

sgi: Set the row boundary to the correct value.

I guess. The change looks ok, not sure why before we would read up to
the linesize, what is z ?

> diff --git a/libavcodec/sgidec.c b/libavcodec/sgidec.c
> index 1145afb..663c314 100644
> --- a/libavcodec/sgidec.c
> +++ b/libavcodec/sgidec.c
> @@ -106,7 +106,7 @@ static int read_rle_sgi(uint8_t *out_buf, SgiState *s)
>              dest_row -= s->linesize;
>              start_offset = bytestream2_get_be32(&g_table);
>              bytestream2_seek(&s->g, start_offset, SEEK_SET);
> -            if (expand_rle_row(s, dest_row + z, FFABS(s->linesize) - z,
> +            if (expand_rle_row(s, dest_row + z, s->width * s->depth,
>                                 s->depth) != s->width) {
>                  return AVERROR_INVALIDDATA;
>              }
> 

_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to