diff options
| author | Aman Gupta <aman@tmm1.net> | 2017-09-25 16:45:53 -0700 |
|---|---|---|
| committer | Aman Gupta <aman@tmm1.net> | 2017-11-13 14:32:27 -0800 |
| commit | dad42bc5a134f60164eab2dfb0892b761603e1e1 (patch) | |
| tree | 952b50c16a678d1f676a8fab322b8e9228c8e290 /libavcodec/h264dec.c | |
| parent | 6515e2834a51e651aa9855d071291eeb8256dbb6 (diff) | |
| download | ffmpeg-dad42bc5a134f60164eab2dfb0892b761603e1e1.tar.gz | |
avcodec/h264, videotoolbox: return AVERROR_INVALIDDATA when no frames are produced
The only reason videotoolbox wouldn't produce frames is if the data fed
to it was invalid, so returning AVERROR_INVALIDDATA makes sense here.
Further, it means AVERROR_EXTERNAL can be used in further commits to signal
fatal VideoToolbox errors, letting the user know that they need to fallback to
another decoder.
Signed-off-by: Aman Gupta <aman@tmm1.net>
Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavcodec/h264dec.c')
| -rw-r--r-- | libavcodec/h264dec.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/h264dec.c b/libavcodec/h264dec.c index aed23c4fba..517f6acc13 100644 --- a/libavcodec/h264dec.c +++ b/libavcodec/h264dec.c @@ -824,7 +824,7 @@ static int output_frame(H264Context *h, AVFrame *dst, H264Picture *srcp) int ret; if (src->format == AV_PIX_FMT_VIDEOTOOLBOX && src->buf[0]->size == 1) - return AVERROR_EXTERNAL; + return AVERROR_INVALIDDATA; ret = av_frame_ref(dst, src); if (ret < 0) |