diff options
| author | Steven Liu <lq@chinaffmpeg.org> | 2022-06-24 13:55:41 +0800 |
|---|---|---|
| committer | Steven Liu <lq@chinaffmpeg.org> | 2022-06-24 15:03:37 +0800 |
| commit | 7c05b7951cb47716230c95744240bc60ec5f9433 (patch) | |
| tree | 0014f97fcacb8ba9ca3de1671f3abac09c0e9211 /libavcodec/audiotoolboxenc.c | |
| parent | ebefca7b0eeec823f823e03ff00aca057d2d60dd (diff) | |
| download | ffmpeg-7c05b7951cb47716230c95744240bc60ec5f9433.tar.gz | |
avcodec/audiotoolboxenc: return external error if encode failed
because the AudioConverterFillComplexBuffer can return 0 or 1 if
success.
so set the ret to 0 it AudioConverterFillComplexBuffer success and
return ret value for success or return AVERROR_EXTERNAL when
AudioConverterFillComplexBuffer failed.
BTW change the error message log level from warning to error.
Reviewed-by: Zhao Zhili <zhilizhao@tencent.com>
Signed-off-by: Steven Liu <lq@chinaffmpeg.org>
Diffstat (limited to 'libavcodec/audiotoolboxenc.c')
| -rw-r--r-- | libavcodec/audiotoolboxenc.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/libavcodec/audiotoolboxenc.c b/libavcodec/audiotoolboxenc.c index f8305ab89b..00293154bf 100644 --- a/libavcodec/audiotoolboxenc.c +++ b/libavcodec/audiotoolboxenc.c @@ -554,11 +554,13 @@ static int ffat_encode(AVCodecContext *avctx, AVPacket *avpkt, avctx->frame_size, &avpkt->pts, &avpkt->duration); + ret = 0; } else if (ret && ret != 1) { - av_log(avctx, AV_LOG_WARNING, "Encode error: %i\n", ret); + av_log(avctx, AV_LOG_ERROR, "Encode error: %i\n", ret); + ret = AVERROR_EXTERNAL; } - return 0; + return ret; } static av_cold void ffat_encode_flush(AVCodecContext *avctx) |