On Tue, May 28, 2013 at 11:05:46AM +0200, Anton Khirnov wrote:

I suspect it should be "all channels were coded" in the commit message.

> ---
>  libavcodec/wavpack.c |    5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/libavcodec/wavpack.c b/libavcodec/wavpack.c
> index 7a14519..0847238 100644
> --- a/libavcodec/wavpack.c
> +++ b/libavcodec/wavpack.c
> @@ -1212,6 +1212,11 @@ static int wavpack_decode_frame(AVCodecContext *avctx, 
> void *data,
>          buf_size -= frame_size;
>      }
>  
> +    if (s->ch_offset != avctx->channels) {
> +        av_log(avctx, AV_LOG_ERROR, "Not enough channels coded in a 
> packet.\n");
> +        return AVERROR_INVALIDDATA;
> +    }
> +
>      *got_frame_ptr = 1;
>  
>      return avpkt->size;
> -- 

LGTM
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to