diff options
author | Jason A. Donenfeld <Jason@zx2c4.com> | 2017-04-06 20:33:30 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2017-05-20 08:28:40 -0400 |
commit | 6240377c574b06c5ef900cab3be2625e482ae8bf (patch) | |
tree | 3c4ee7b33e2c5c1b7513ab835c05b773067a9b33 | |
parent | 1c5d8b377e58ba7d56435b4e1e355aef7999ef75 (diff) |
padata: free correct variable
commit 07a77929ba672d93642a56dc2255dd21e6e2290b upstream.
The author meant to free the variable that was just allocated, instead
of the one that failed to be allocated, but made a simple typo. This
patch rectifies that.
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | kernel/padata.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/kernel/padata.c b/kernel/padata.c index b4a3c0ae649b..e4a8f8d9b31a 100644 --- a/kernel/padata.c +++ b/kernel/padata.c | |||
@@ -358,7 +358,7 @@ static int padata_setup_cpumasks(struct parallel_data *pd, | |||
358 | 358 | ||
359 | cpumask_and(pd->cpumask.pcpu, pcpumask, cpu_online_mask); | 359 | cpumask_and(pd->cpumask.pcpu, pcpumask, cpu_online_mask); |
360 | if (!alloc_cpumask_var(&pd->cpumask.cbcpu, GFP_KERNEL)) { | 360 | if (!alloc_cpumask_var(&pd->cpumask.cbcpu, GFP_KERNEL)) { |
361 | free_cpumask_var(pd->cpumask.cbcpu); | 361 | free_cpumask_var(pd->cpumask.pcpu); |
362 | return -ENOMEM; | 362 | return -ENOMEM; |
363 | } | 363 | } |
364 | 364 | ||