On 4 March 2016 at 11:55, Jon Hunter <jonathanh@xxxxxxxxxx> wrote: > 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@xxxxxxxxxx> Acked-by: Ulf Hansson <ulf.hansson@xxxxxxxxxx> Kind regards Uffe > --- > drivers/base/power/domain.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c > index ea9f2aa3fc33..5bdb42bf40f4 100644 > --- a/drivers/base/power/domain.c > +++ b/drivers/base/power/domain.c > @@ -1686,6 +1686,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 */ > -- > 2.1.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
