On Wed, Oct 26, 2011 at 04:18:34PM -0400, Justin Ruggles wrote:
> ---
> libavcodec/apedec.c | 5 +++++
> 1 files changed, 5 insertions(+), 0 deletions(-)
>
> diff --git a/libavcodec/apedec.c b/libavcodec/apedec.c
> index 57381b3..99f1eae 100644
> --- a/libavcodec/apedec.c
> +++ b/libavcodec/apedec.c
> @@ -26,6 +26,7 @@
> #include "get_bits.h"
> #include "bytestream.h"
> #include "libavutil/audioconvert.h"
> +#include "libavutil/avassert.h"
>
> /**
> * @file
> @@ -834,6 +835,10 @@ static int ape_decode_frame(AVCodecContext *avctx,
> return AVERROR(EINVAL);
> }
>
> + /* this should never be negative, but bad things will happen if it is, so
> + check it just to make sure. */
> + av_assert0(s->samples >= 0);
> +
> if(!s->samples){
> uint32_t offset;
> void *tmp_data;
> --
LGTM
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
