[FFmpeg-devel] [PATCH] tests/checkasm: Remove check on linux perf fd in uninit

Zhao Zhili quinkblack at foxmail.com
Wed Jun 12 20:35:53 EEST 2024


From: Zhao Zhili <zhilizhao at tencent.com>

The check should be >= 0, not > 0. The check itself is redundant
since uninit only being called after init is success.
---
 tests/checkasm/checkasm.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tests/checkasm/checkasm.c b/tests/checkasm/checkasm.c
index 28237b4d25..bbcc90f91f 100644
--- a/tests/checkasm/checkasm.c
+++ b/tests/checkasm/checkasm.c
@@ -814,8 +814,7 @@ static int bench_init(void)
 static void bench_uninit(void)
 {
 #if CONFIG_LINUX_PERF
-    if (state.sysfd > 0)
-        close(state.sysfd);
+    close(state.sysfd);
 #endif
 }
 
-- 
2.42.0



More information about the ffmpeg-devel mailing list