diff options
author | Li Zefan <lizf@cn.fujitsu.com> | 2008-11-04 03:20:23 -0500 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2008-11-04 04:25:13 -0500 |
commit | faa2f98f856e89d1afb6e4a91707284d242e816e (patch) | |
tree | 3674a562130b89bf127c188102af6335cc5c4469 /kernel | |
parent | a17e2260926f681a0eb983c1e3cb859ba2064bce (diff) |
sched: add sanity check in partition_sched_domains()
Impact: cleanup, add debug check
It's wrong to make dattr_new = NULL if doms_new == NULL, it introduces
memory leak if dattr_new != NULL. Fortunately dattr_new is always NULL
in this case. So remove the code and add a sanity check.
Signed-off-by: Li Zefan <lizf@cn.fujitsu.com>
Acked-by: Peter Zijlstra <a.p.zijlstra@chello.nl>
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel')
-rw-r--r-- | kernel/sched.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/sched.c b/kernel/sched.c index 3cb94fad33ca..213cad5e50aa 100644 --- a/kernel/sched.c +++ b/kernel/sched.c | |||
@@ -7767,7 +7767,7 @@ match1: | |||
7767 | ndoms_cur = 0; | 7767 | ndoms_cur = 0; |
7768 | doms_new = &fallback_doms; | 7768 | doms_new = &fallback_doms; |
7769 | cpus_andnot(doms_new[0], cpu_online_map, cpu_isolated_map); | 7769 | cpus_andnot(doms_new[0], cpu_online_map, cpu_isolated_map); |
7770 | dattr_new = NULL; | 7770 | WARN_ON_ONCE(dattr_new); |
7771 | } | 7771 | } |
7772 | 7772 | ||
7773 | /* Build new domains */ | 7773 | /* Build new domains */ |