On 12/15/16, Michael Niedermayer <mich...@niedermayer.cc> wrote:
> Most decoders return the amount of data used.
> This is more consistent
>
> Signed-off-by: Michael Niedermayer <mich...@niedermayer.cc>
> ---
>  libavcodec/smvjpegdec.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/libavcodec/smvjpegdec.c b/libavcodec/smvjpegdec.c
> index e319e5781b..018e135b30 100644
> --- a/libavcodec/smvjpegdec.c
> +++ b/libavcodec/smvjpegdec.c
> @@ -200,9 +200,11 @@ static int smvjpeg_decode_frame(AVCodecContext *avctx,
> void *data, int *data_siz
>              s->picture[1]->linesize[i] = mjpeg_data->linesize[i];
>
>          ret = av_frame_ref(data, s->picture[1]);
> +        if (ret < 0)
> +            return ret;
>      }
>
> -    return ret;
> +    return avpkt->size;
>  }
>
>  static const AVClass smvjpegdec_class = {
> --
> 2.11.0
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>

lgtm
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to