[FFmpeg-cvslog] avutil/frame: Return 0 on success in av_frame_ref()

Andreas Rheinhardt git at videolan.org
Mon Apr 5 19:39:11 EEST 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at outlook.com> | Thu Apr  1 23:04:19 2021 +0200| [416cc012f6203c7162079ae5b14930ada9189ac3] | committer: Andreas Rheinhardt

avutil/frame: Return 0 on success in av_frame_ref()

av_frame_copy() is allowed to return values >= 0 on success, whereas
the documentation of av_frame_ref() states that the return value is 0 on
success. Ergo the latter must not just return the former's return value.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=416cc012f6203c7162079ae5b14930ada9189ac3
---

 libavutil/frame.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavutil/frame.c b/libavutil/frame.c
index 31a2117b82..fefb2b69c0 100644
--- a/libavutil/frame.c
+++ b/libavutil/frame.c
@@ -469,7 +469,7 @@ int av_frame_ref(AVFrame *dst, const AVFrame *src)
         if (ret < 0)
             goto fail;
 
-        return ret;
+        return 0;
     }
 
     /* ref the buffers */



More information about the ffmpeg-cvslog mailing list