On 2013-01-08 15:37:07 +0100, Anton Khirnov wrote:
> ---
>  libavcodec/nuv.c |   18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/libavcodec/nuv.c b/libavcodec/nuv.c
> index f7d348a..67fbf5f 100644
> --- a/libavcodec/nuv.c
> +++ b/libavcodec/nuv.c
> @@ -28,6 +28,7 @@
>  #include "libavutil/imgutils.h"
>  #include "avcodec.h"
>  #include "dsputil.h"
> +#include "internal.h"
>  #include "rtjpeg.h"
>  
>  typedef struct {
> @@ -221,12 +222,10 @@ static int decode_frame(AVCodecContext *avctx, void 
> *data, int *got_frame,
>          buf_size -= RTJPEG_HEADER_SIZE;
>      }
>  
> -    if (keyframe && c->pic.data[0])
> -        avctx->release_buffer(avctx, &c->pic);
> -    c->pic.reference    = 3;
> -    c->pic.buffer_hints = FF_BUFFER_HINTS_VALID    | 
> FF_BUFFER_HINTS_READABLE |
> -                          FF_BUFFER_HINTS_PRESERVE | 
> FF_BUFFER_HINTS_REUSABLE;
> -    result = avctx->reget_buffer(avctx, &c->pic);
> +    if (keyframe)
> +        av_frame_unref(&c->pic);
> +
> +    result = ff_reget_buffer(avctx, &c->pic);
>      if (result < 0) {
>          av_log(avctx, AV_LOG_ERROR, "get_buffer() failed\n");
>          return result;
> @@ -263,7 +262,9 @@ static int decode_frame(AVCodecContext *avctx, void 
> *data, int *got_frame,
>          return AVERROR_INVALIDDATA;
>      }
>  
> -    *picture   = c->pic;
> +    if ((result = av_frame_ref(picture, &c->pic)) < 0)
> +        return result;
> +
>      *got_frame = 1;
>      return orig_size;
>  }
> @@ -298,8 +299,7 @@ static av_cold int decode_end(AVCodecContext *avctx)
>      NuvContext *c = avctx->priv_data;
>  
>      av_freep(&c->decomp_buf);
> -    if (c->pic.data[0])
> -        avctx->release_buffer(avctx, &c->pic);
> +    av_frame_unref(&c->pic);
>  
>      return 0;
>  }

looks ok

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

Reply via email to