diff options
| author | Rui Qi <qirui.001@bytedance.com> | 2026-06-04 11:32:08 +0300 |
|---|---|---|
| committer | Petr Mladek <pmladek@suse.com> | 2026-06-09 13:38:33 +0300 |
| commit | 7203abb3f8bd61e021cce076dafdda7bcfd9f2c2 (patch) | |
| tree | c3b707cdfbbaf5825233e9fca0dacdfb7ef27500 | |
| parent | ed8f317671d9a6bd1daf2f0cb152e126eae2bf10 (diff) | |
| download | linux-7203abb3f8bd61e021cce076dafdda7bcfd9f2c2.tar.xz | |
selftests/livepatch: fix resource leak in test_klp_syscall init error path
In livepatch_init(), if klp_enable_patch() fails, the previously
created kobject and sysfs file are never cleaned up, causing a
resource leak. Capture the return value and add proper cleanup
on the error path.
Signed-off-by: Rui Qi <qirui.001@bytedance.com>
Acked-by: Miroslav Benes <mbenes@suse.cz>
Reviewed-by: Petr Mladek <pmladek@suse.com>
Link: https://patch.msgid.link/20260604083208.1071428-1-qirui.001@bytedance.com
Signed-off-by: Petr Mladek <pmladek@suse.com>
| -rw-r--r-- | tools/testing/selftests/livepatch/test_modules/test_klp_syscall.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/testing/selftests/livepatch/test_modules/test_klp_syscall.c b/tools/testing/selftests/livepatch/test_modules/test_klp_syscall.c index 0630ffd9d9a1..08aacc0e14de 100644 --- a/tools/testing/selftests/livepatch/test_modules/test_klp_syscall.c +++ b/tools/testing/selftests/livepatch/test_modules/test_klp_syscall.c @@ -109,7 +109,11 @@ static int livepatch_init(void) */ npids = npids_pending; - return klp_enable_patch(&patch); + ret = klp_enable_patch(&patch); + if (ret) + kobject_put(klp_kobj); + + return ret; } static void livepatch_exit(void) |