* [PATCH] cpufreq: scmi: fix use-after-free in scmi_cpufreq_exit()
@ 2019-02-16 16:31 Yangtao Li
2019-02-18 4:53 ` Viresh Kumar
2019-02-18 11:04 ` Sudeep Holla
0 siblings, 2 replies; 4+ messages in thread
From: Yangtao Li @ 2019-02-16 16:31 UTC (permalink / raw)
To: sudeep.holla, rjw, viresh.kumar
Cc: Yangtao Li, linux-kernel, linux-arm-kernel, linux-pm
This issue was detected with the help of Coccinelle. So
change the order of function calls to fix it.
Fixes: 1690d8bb91e37 (cpufreq: scpi/scmi: Fix freeing of dynamic OPPs)
Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
---
drivers/cpufreq/scmi-cpufreq.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpufreq/scmi-cpufreq.c b/drivers/cpufreq/scmi-cpufreq.c
index 242c3370544e..9ed46d188cb5 100644
--- a/drivers/cpufreq/scmi-cpufreq.c
+++ b/drivers/cpufreq/scmi-cpufreq.c
@@ -187,8 +187,8 @@ static int scmi_cpufreq_exit(struct cpufreq_policy *policy)
cpufreq_cooling_unregister(priv->cdev);
dev_pm_opp_free_cpufreq_table(priv->cpu_dev, &policy->freq_table);
- kfree(priv);
dev_pm_opp_remove_all_dynamic(priv->cpu_dev);
+ kfree(priv);
return 0;
}
--
2.17.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] cpufreq: scmi: fix use-after-free in scmi_cpufreq_exit()
2019-02-16 16:31 [PATCH] cpufreq: scmi: fix use-after-free in scmi_cpufreq_exit() Yangtao Li
@ 2019-02-18 4:53 ` Viresh Kumar
2019-02-19 10:18 ` Rafael J. Wysocki
2019-02-18 11:04 ` Sudeep Holla
1 sibling, 1 reply; 4+ messages in thread
From: Viresh Kumar @ 2019-02-18 4:53 UTC (permalink / raw)
To: Yangtao Li; +Cc: linux-pm, rjw, linux-kernel, linux-arm-kernel, sudeep.holla
On 16-02-19, 11:31, Yangtao Li wrote:
> This issue was detected with the help of Coccinelle. So
> change the order of function calls to fix it.
>
> Fixes: 1690d8bb91e37 (cpufreq: scpi/scmi: Fix freeing of dynamic OPPs)
>
> Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> ---
> drivers/cpufreq/scmi-cpufreq.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/cpufreq/scmi-cpufreq.c b/drivers/cpufreq/scmi-cpufreq.c
> index 242c3370544e..9ed46d188cb5 100644
> --- a/drivers/cpufreq/scmi-cpufreq.c
> +++ b/drivers/cpufreq/scmi-cpufreq.c
> @@ -187,8 +187,8 @@ static int scmi_cpufreq_exit(struct cpufreq_policy *policy)
>
> cpufreq_cooling_unregister(priv->cdev);
> dev_pm_opp_free_cpufreq_table(priv->cpu_dev, &policy->freq_table);
> - kfree(priv);
> dev_pm_opp_remove_all_dynamic(priv->cpu_dev);
> + kfree(priv);
>
> return 0;
> }
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
@Rafael: Please pick it up for 5.0-rc8 as the bug was introduced
during 5.0 cycle only.
The patch it fixes had this tag:
Cc: 4.20 <stable@vger.kernel.org> # v4.20
And so will get applied to 4.20.N, I guess we need to mark this patch
as well for stable then.
--
viresh
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] cpufreq: scmi: fix use-after-free in scmi_cpufreq_exit()
2019-02-16 16:31 [PATCH] cpufreq: scmi: fix use-after-free in scmi_cpufreq_exit() Yangtao Li
2019-02-18 4:53 ` Viresh Kumar
@ 2019-02-18 11:04 ` Sudeep Holla
1 sibling, 0 replies; 4+ messages in thread
From: Sudeep Holla @ 2019-02-18 11:04 UTC (permalink / raw)
To: Yangtao Li
Cc: linux-pm, viresh.kumar, rjw, linux-kernel, Sudeep Holla,
linux-arm-kernel
On Sat, Feb 16, 2019 at 11:31:48AM -0500, Yangtao Li wrote:
> This issue was detected with the help of Coccinelle. So
> change the order of function calls to fix it.
>
> Fixes: 1690d8bb91e37 (cpufreq: scpi/scmi: Fix freeing of dynamic OPPs)
>
Acked-by: Sudeep Holla <sudeep.holla@arm.com>
--
Regards,
Sudeep
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] cpufreq: scmi: fix use-after-free in scmi_cpufreq_exit()
2019-02-18 4:53 ` Viresh Kumar
@ 2019-02-19 10:18 ` Rafael J. Wysocki
0 siblings, 0 replies; 4+ messages in thread
From: Rafael J. Wysocki @ 2019-02-19 10:18 UTC (permalink / raw)
To: Viresh Kumar
Cc: Yangtao Li, linux-pm, linux-kernel, linux-arm-kernel, sudeep.holla
On Monday, February 18, 2019 5:53:30 AM CET Viresh Kumar wrote:
> On 16-02-19, 11:31, Yangtao Li wrote:
> > This issue was detected with the help of Coccinelle. So
> > change the order of function calls to fix it.
> >
> > Fixes: 1690d8bb91e37 (cpufreq: scpi/scmi: Fix freeing of dynamic OPPs)
> >
> > Signed-off-by: Yangtao Li <tiny.windzz@gmail.com>
> > ---
> > drivers/cpufreq/scmi-cpufreq.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/cpufreq/scmi-cpufreq.c b/drivers/cpufreq/scmi-cpufreq.c
> > index 242c3370544e..9ed46d188cb5 100644
> > --- a/drivers/cpufreq/scmi-cpufreq.c
> > +++ b/drivers/cpufreq/scmi-cpufreq.c
> > @@ -187,8 +187,8 @@ static int scmi_cpufreq_exit(struct cpufreq_policy *policy)
> >
> > cpufreq_cooling_unregister(priv->cdev);
> > dev_pm_opp_free_cpufreq_table(priv->cpu_dev, &policy->freq_table);
> > - kfree(priv);
> > dev_pm_opp_remove_all_dynamic(priv->cpu_dev);
> > + kfree(priv);
> >
> > return 0;
> > }
>
> Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
>
> @Rafael: Please pick it up for 5.0-rc8 as the bug was introduced
> during 5.0 cycle only.
>
> The patch it fixes had this tag:
>
> Cc: 4.20 <stable@vger.kernel.org> # v4.20
>
> And so will get applied to 4.20.N, I guess we need to mark this patch
> as well for stable then.
Done now, thanks!
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2019-02-19 10:20 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-02-16 16:31 [PATCH] cpufreq: scmi: fix use-after-free in scmi_cpufreq_exit() Yangtao Li
2019-02-18 4:53 ` Viresh Kumar
2019-02-19 10:18 ` Rafael J. Wysocki
2019-02-18 11:04 ` Sudeep Holla
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).