diff options
author | Sean Fu <fxinrong@gmail.com> | 2017-09-11 08:33:21 +0800 |
---|---|---|
committer | Thomas Gleixner <tglx@linutronix.de> | 2017-09-25 09:36:16 +0200 |
commit | 7d7099433d9eaaa5a989a55f1fa354c16a3ad297 (patch) | |
tree | 256e2478d50bb3f68bae7892f763c2843d7d69d3 /arch | |
parent | a3c4fb7c9c2ebfd50b8c60f6c069932bb319bc37 (diff) | |
download | linux-7d7099433d9eaaa5a989a55f1fa354c16a3ad297.tar.gz linux-7d7099433d9eaaa5a989a55f1fa354c16a3ad297.tar.bz2 linux-7d7099433d9eaaa5a989a55f1fa354c16a3ad297.zip |
x86/sysfs: Fix off-by-one error in loop termination
An off-by-one error in loop terminantion conditions in
create_setup_data_nodes() will lead to memory leak when
create_setup_data_node() failed.
Signed-off-by: Sean Fu <fxinrong@gmail.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Link: http://lkml.kernel.org/r/1505090001-1157-1-git-send-email-fxinrong@gmail.com
Diffstat (limited to 'arch')
-rw-r--r-- | arch/x86/kernel/ksysfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/x86/kernel/ksysfs.c b/arch/x86/kernel/ksysfs.c index 4b0592ca9e47..8c1cc08f514f 100644 --- a/arch/x86/kernel/ksysfs.c +++ b/arch/x86/kernel/ksysfs.c @@ -299,7 +299,7 @@ static int __init create_setup_data_nodes(struct kobject *parent) return 0; out_clean_nodes: - for (j = i - 1; j > 0; j--) + for (j = i - 1; j >= 0; j--) cleanup_setup_data_node(*(kobjp + j)); kfree(kobjp); out_setup_data_kobj: |