25. 9. 9. 19:29에 Dan Carpenter 이(가) 쓴 글:
> The put_device() function calls devfreq_event_release_edev() which frees
> "evdev". Calling kfree() again is a double free.
>
> Fixes: 430a1845c804 ("PM / devfreq: Fix memory leak in devfreq_event_add_edev()")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
> ---
> drivers/devfreq/devfreq-event.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/drivers/devfreq/devfreq-event.c b/drivers/devfreq/devfreq-event.c
> index 34406c52b845..70219099c604 100644
> --- a/drivers/devfreq/devfreq-event.c
> +++ b/drivers/devfreq/devfreq-event.c
> @@ -328,7 +328,6 @@ struct devfreq_event_dev *devfreq_event_add_edev(struct device *dev,
> ret = device_register(&edev->dev);
> if (ret < 0) {
> put_device(&edev->dev);
> - kfree(edev);
> return ERR_PTR(ret);
> }
> dev_set_drvdata(&edev->dev, edev);
Firstly, I'm sorry that it my wrong review of patch[1].
[1] https://patchwork.kernel.org/project/linux-pm/patch/20250907113302.3353584-1-kaushlendra.kumar@xxxxxxxxx/
As you mentioned, the above patch[1] doesn't be necessary.
Instead of applying your patch, I'll drop the patch[1].
I'm sorry to make the confusion.
--
Best Regards,
Samsung Electronics
Chanwoo Choi
