diff options
| author | Li Zefan <lizf@cn.fujitsu.com> | 2008-11-04 16:20:23 +0800 |
|---|---|---|
| committer | Ingo Molnar <mingo@elte.hu> | 2008-11-04 10:25:13 +0100 |
| commit | faa2f98f856e89d1afb6e4a91707284d242e816e (patch) | |
| tree | 3674a562130b89bf127c188102af6335cc5c4469 /kernel | |
| parent | a17e2260926f681a0eb983c1e3cb859ba2064bce (diff) | |
| download | linux-0-day-faa2f98f856e89d1afb6e4a91707284d242e816e.tar.gz linux-0-day-faa2f98f856e89d1afb6e4a91707284d242e816e.tar.xz | |
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 3cb94fad33cae..213cad5e50aa4 100644 --- a/kernel/sched.c +++ b/kernel/sched.c @@ -7767,7 +7767,7 @@ match1: ndoms_cur = 0; doms_new = &fallback_doms; cpus_andnot(doms_new[0], cpu_online_map, cpu_isolated_map); - dattr_new = NULL; + WARN_ON_ONCE(dattr_new); } /* Build new domains */ |