From: Emil Velikov <emil.l.velikov@gmail.com>
Date: Fri, 13 Jun 2025 16:45:34 +0000 (+0100)
Subject: testsuite: remove exit() calls from main()
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=41b1798768c1d65796b9d9092a60541c49d5daa1;p=thirdparty%2Fkmod.git

testsuite: remove exit() calls from main()

There is no particular reason for us to use exit(), so just avoid it.

As result, the only instances using of exit() are within the child
process and our cleanup is complete :-)

Signed-off-by: Emil Velikov <emil.l.velikov@gmail.com>
Link: https://github.com/kmod-project/kmod/pull/371
Signed-off-by: Lucas De Marchi <lucas.de.marchi@gmail.com>
---

diff --git a/testsuite/testsuite.h b/testsuite/testsuite.h
index 9b7a5620..ae17a321 100644
--- a/testsuite/testsuite.h
+++ b/testsuite/testsuite.h
@@ -161,7 +161,7 @@ int test_run(const struct test *t);
 			t = test_find(__start_kmod_tests, __stop_kmod_tests, argv[arg]); \
 			if (t == NULL) {                                                 \
 				fprintf(stderr, "could not find test %s\n", argv[arg]);  \
-				exit(EXIT_FAILURE);                                      \
+				return EXIT_FAILURE;                                     \
 			}                                                                \
                                                                                          \
 			return test_run(t);                                              \
@@ -172,5 +172,5 @@ int test_run(const struct test *t);
 				ret = EXIT_FAILURE;                                      \
 		}                                                                        \
                                                                                          \
-		exit(ret);                                                               \
+		return ret;                                                              \
 	}