diff options
| author | Peng Fan <peng.fan@nxp.com> | 2019-02-20 13:32:55 +0000 |
|---|---|---|
| committer | Dennis Zhou <dennis@kernel.org> | 2019-02-23 14:36:53 -0800 |
| commit | 2de7852fe9096e92c5f10faa472550a2a7121cea (patch) | |
| tree | cde1e86e79d9b743875d9cf9dbb738d3d6a83bfb | |
| parent | a3b22b9f11d9fbc48b0291ea92259a5a810e9438 (diff) | |
percpu: use nr_groups as check condition
group_cnt array is defined with NR_CPUS entries, but normally
nr_groups will not reach up to NR_CPUS. So there is no issue
to the current code.
Checking other parts of pcpu_build_alloc_info, use nr_groups as
check condition, so make it consistent to use 'group < nr_groups'
as for loop check. In case we do have nr_groups equals with NR_CPUS,
we could also avoid memory access out of bounds.
Signed-off-by: Peng Fan <peng.fan@nxp.com>
Signed-off-by: Dennis Zhou <dennis@kernel.org>
| -rw-r--r-- | mm/percpu.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/mm/percpu.c b/mm/percpu.c index db86282fd024..c5c750781628 100644 --- a/mm/percpu.c +++ b/mm/percpu.c @@ -2384,7 +2384,7 @@ static struct pcpu_alloc_info * __init pcpu_build_alloc_info( ai->atom_size = atom_size; ai->alloc_size = alloc_size; - for (group = 0, unit = 0; group_cnt[group]; group++) { + for (group = 0, unit = 0; group < nr_groups; group++) { struct pcpu_group_info *gi = &ai->groups[group]; /* |