From 416cc012f6203c7162079ae5b14930ada9189ac3 Mon Sep 17 00:00:00 2001 From: Andreas Rheinhardt Date: Thu, 1 Apr 2021 23:04:19 +0200 Subject: [PATCH] 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 --- 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 */