Re: [PATCH next] PM / devfreq: Fix double free in devfreq_event_add_edev()

[Chanwoo Choi <chanwoo@kernel.org>]

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@linaro.org>
> ---
>  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@intel.com/

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