diff options
author | Jon Hunter <jonathanh@nvidia.com> | 2016-03-04 10:55:15 +0000 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2016-03-09 00:41:07 +0100 |
commit | 41795a8a3cff8e4ba54236ca16c3814ba9cd7f39 (patch) | |
tree | ba2adcb9e606cee93dbac2febc562805702e0fcd /drivers/base/power | |
parent | beda5fc1ff9b527059290a97b672d2ee0eb7b92f (diff) | |
download | op-kernel-dev-41795a8a3cff8e4ba54236ca16c3814ba9cd7f39.zip op-kernel-dev-41795a8a3cff8e4ba54236ca16c3814ba9cd7f39.tar.gz |
PM / Domains: Fix potential NULL pointer dereference
In the function of_genpd_get_from_provider(), we never check to see if
the argument 'genpdspec' is NULL before dereferencing it. Add error
checking to handle any NULL pointers.
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
Acked-by: Ulf Hansson <ulf.hansson@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/base/power')
-rw-r--r-- | drivers/base/power/domain.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index 79f5d39..56705b5 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c @@ -1702,6 +1702,9 @@ struct generic_pm_domain *of_genpd_get_from_provider( struct generic_pm_domain *genpd = ERR_PTR(-ENOENT); struct of_genpd_provider *provider; + if (!genpdspec) + return ERR_PTR(-EINVAL); + mutex_lock(&of_genpd_mutex); /* Check if we have such a provider in our array */ |