On 10/17/2012 11:31 PM, Reinhard Tartler wrote:
> From: Michael Niedermayer <michae...@gmx.at>
>
> Fixes Ticket1633
>
> Found-by: Piotr Bandurski <ami_st...@o2.pl>
> Signed-off-by: Michael Niedermayer <michae...@gmx.at>
> (cherry picked from commit f0896a6bd94e5b45447c7d640c8e8aa95d860d7a)
>
> Signed-off-by: Reinhard Tartler <siret...@tauware.de>
> ---
> libavcodec/sp5xdec.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/sp5xdec.c b/libavcodec/sp5xdec.c
> index 29b0059..67270da 100644
> --- a/libavcodec/sp5xdec.c
> +++ b/libavcodec/sp5xdec.c
> @@ -72,7 +72,7 @@ static int sp5x_decode_frame(AVCodecContext *avctx,
> for (i = 2; i < buf_size-2 && j < buf_size+1024-2; i++)
> recoded[j++] = buf[i];
> else
> - for (i = 14; i < buf_size && j < buf_size+1024-2; i++)
> + for (i = 14; i < buf_size && j < buf_size+1024-3; i++)
> {
> recoded[j++] = buf[i];
> if (buf[i] == 0xff)
>
While at it give the line some space between operators.
I wonder what about
> for (i = 2; i < buf_size-2 && j < buf_size+1024-2; i++)
and what 2 and 3 exactly mean...
lu
_______________________________________________
libav-devel mailing list
libav-devel@libav.org
https://lists.libav.org/mailman/listinfo/libav-devel
