On 11.07.2015 13:08, Luca Barbato wrote:
> Properly report the sample rate as invalid
> 
> CC: [email protected]
> ---
>  libavcodec/g726.c | 7 +++++--
>  1 file changed, 5 insertions(+), 2 deletions(-)
> 
> diff --git a/libavcodec/g726.c b/libavcodec/g726.c
> index 9ad91f2..b877687 100644
> --- a/libavcodec/g726.c
> +++ b/libavcodec/g726.c
> @@ -23,7 +23,6 @@
>   */
>  #include <limits.h>
>  
> -#include "libavutil/avassert.h"
>  #include "libavutil/channel_layout.h"
>  #include "libavutil/opt.h"
>  #include "avcodec.h"
> @@ -315,7 +314,11 @@ static av_cold int g726_encode_init(AVCodecContext 
> *avctx)
>                 "Resample or reduce the compliance level.\n");
>          return AVERROR(EINVAL);
>      }
> -    av_assert0(avctx->sample_rate > 0);
> +    if (avctx->sample_rate <= 0) {
> +        av_log(avctx, AV_LOG_ERROR, "Invalid sample rate %d\n",
> +               avctx->sample_rate);
> +        return AVERROR(EINVAL);
> +    }
>  
>      if(avctx->channels != 1){
>          av_log(avctx, AV_LOG_ERROR, "Only mono is supported\n");
> 

Looks good.

Best regards,
Andreas
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to