* [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister
@ 2014-10-20 8:27 Yao Dongdong
2014-10-20 12:33 ` Eduardo Valentin
0 siblings, 1 reply; 5+ messages in thread
From: Yao Dongdong @ 2014-10-20 8:27 UTC (permalink / raw)
To: Zhang Rui, Eduardo Valentin; +Cc: linux-pm, LKML, yaodongdong
Signed-off-by:yaodongdong@huawei.com
---
drivers/thermal/thermal_core.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
index 71b0ec0..5b7d466 100644
--- a/drivers/thermal/thermal_core.c
+++ b/drivers/thermal/thermal_core.c
@@ -1574,6 +1574,7 @@ struct thermal_zone_device *thermal_zone_device_register(const char *type,
unregister:
release_idr(&thermal_tz_idr, &thermal_idr_lock, tz->id);
device_unregister(&tz->device);
+ kfree(tz);
return ERR_PTR(result);
}
EXPORT_SYMBOL_GPL(thermal_zone_device_register);
--
1.8.0.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister
2014-10-20 8:27 [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister Yao Dongdong
@ 2014-10-20 12:33 ` Eduardo Valentin
2014-11-03 18:17 ` R, Durgadoss
0 siblings, 1 reply; 5+ messages in thread
From: Eduardo Valentin @ 2014-10-20 12:33 UTC (permalink / raw)
To: Yao Dongdong; +Cc: Zhang Rui, linux-pm, LKML
[-- Attachment #1: Type: text/plain, Size: 840 bytes --]
Hello,
On Mon, Oct 20, 2014 at 04:27:36PM +0800, Yao Dongdong wrote:
> Signed-off-by:yaodongdong@huawei.com
Acked-by: Eduardo Valentin <edubezval@gmail.com>
Rui, would you take care of this?
>
> ---
> drivers/thermal/thermal_core.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
> index 71b0ec0..5b7d466 100644
> --- a/drivers/thermal/thermal_core.c
> +++ b/drivers/thermal/thermal_core.c
> @@ -1574,6 +1574,7 @@ struct thermal_zone_device *thermal_zone_device_register(const char *type,
> unregister:
> release_idr(&thermal_tz_idr, &thermal_idr_lock, tz->id);
> device_unregister(&tz->device);
> + kfree(tz);
> return ERR_PTR(result);
> }
> EXPORT_SYMBOL_GPL(thermal_zone_device_register);
> --
> 1.8.0.1
>
>
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* RE: [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister
2014-10-20 12:33 ` Eduardo Valentin
@ 2014-11-03 18:17 ` R, Durgadoss
2014-11-03 18:35 ` Eduardo Valentin
0 siblings, 1 reply; 5+ messages in thread
From: R, Durgadoss @ 2014-11-03 18:17 UTC (permalink / raw)
To: Eduardo Valentin, Yao Dongdong; +Cc: Zhang, Rui, linux-pm, LKML
>-----Original Message-----
>From: linux-pm-owner@vger.kernel.org [mailto:linux-pm-
>owner@vger.kernel.org] On Behalf Of Eduardo Valentin
>Sent: Monday, October 20, 2014 6:04 PM
>To: Yao Dongdong
>Cc: Zhang, Rui; linux-pm@vger.kernel.org; LKML
>Subject: Re: [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister
>
>Hello,
>
>On Mon, Oct 20, 2014 at 04:27:36PM +0800, Yao Dongdong wrote:
>> Signed-off-by:yaodongdong@huawei.com
>
>Acked-by: Eduardo Valentin <edubezval@gmail.com>
>
>Rui, would you take care of this?
>
If I remember it right, this 'tz' is freed in the thermal_release()
function, during device_unregister().
It is similar in all other functions in thermal_core.c
So, Yao, Did you really test this patch ?
And did not see any crashes ?
Thanks,
Durga
>>
>> ---
>> drivers/thermal/thermal_core.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
>> index 71b0ec0..5b7d466 100644
>> --- a/drivers/thermal/thermal_core.c
>> +++ b/drivers/thermal/thermal_core.c
>> @@ -1574,6 +1574,7 @@ struct thermal_zone_device
>*thermal_zone_device_register(const char *type,
>> unregister:
>> release_idr(&thermal_tz_idr, &thermal_idr_lock, tz->id);
>> device_unregister(&tz->device);
>> + kfree(tz);
>> return ERR_PTR(result);
>> }
>> EXPORT_SYMBOL_GPL(thermal_zone_device_register);
>> --
>> 1.8.0.1
>>
>>
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister
2014-11-03 18:17 ` R, Durgadoss
@ 2014-11-03 18:35 ` Eduardo Valentin
2014-11-04 2:01 ` Yao Dongdong
0 siblings, 1 reply; 5+ messages in thread
From: Eduardo Valentin @ 2014-11-03 18:35 UTC (permalink / raw)
To: R, Durgadoss; +Cc: Yao Dongdong, Zhang, Rui, linux-pm, LKML
[-- Attachment #1: Type: text/plain, Size: 2053 bytes --]
Hello,
On Mon, Nov 03, 2014 at 06:17:37PM +0000, R, Durgadoss wrote:
> >-----Original Message-----
> >From: linux-pm-owner@vger.kernel.org [mailto:linux-pm-
> >owner@vger.kernel.org] On Behalf Of Eduardo Valentin
> >Sent: Monday, October 20, 2014 6:04 PM
> >To: Yao Dongdong
> >Cc: Zhang, Rui; linux-pm@vger.kernel.org; LKML
> >Subject: Re: [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister
> >
> >Hello,
> >
> >On Mon, Oct 20, 2014 at 04:27:36PM +0800, Yao Dongdong wrote:
> >> Signed-off-by:yaodongdong@huawei.com
> >
> >Acked-by: Eduardo Valentin <edubezval@gmail.com>
> >
> >Rui, would you take care of this?
> >
>
> If I remember it right, this 'tz' is freed in the thermal_release()
> function, during device_unregister().
>
> It is similar in all other functions in thermal_core.c
>
> So, Yao, Did you really test this patch ?
> And did not see any crashes ?
>
Yao, reading the patch change carefully, now I see that Durga is correct
here.
> Thanks,
> Durga
>
> >>
> >> ---
> >> drivers/thermal/thermal_core.c | 1 +
> >> 1 file changed, 1 insertion(+)
> >>
> >> diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
> >> index 71b0ec0..5b7d466 100644
> >> --- a/drivers/thermal/thermal_core.c
> >> +++ b/drivers/thermal/thermal_core.c
> >> @@ -1574,6 +1574,7 @@ struct thermal_zone_device
> >*thermal_zone_device_register(const char *type,
> >> unregister:
> >> release_idr(&thermal_tz_idr, &thermal_idr_lock, tz->id);
> >> device_unregister(&tz->device);
> >> + kfree(tz);
The device unregister is called above your kfree call, which will cause
the thermal_release to be called. Did you test the code for double kfree
calls? Your patch probably inserts a memory corruption.
I will revert your patch from my tree until you provide an answer about
your testing.
Thanks,
Eduardo Valentin
> >> return ERR_PTR(result);
> >> }
> >> EXPORT_SYMBOL_GPL(thermal_zone_device_register);
> >> --
> >> 1.8.0.1
> >>
> >>
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister
2014-11-03 18:35 ` Eduardo Valentin
@ 2014-11-04 2:01 ` Yao Dongdong
0 siblings, 0 replies; 5+ messages in thread
From: Yao Dongdong @ 2014-11-04 2:01 UTC (permalink / raw)
To: Eduardo Valentin, R, Durgadoss; +Cc: Zhang, Rui, linux-pm, LKML
On 2014/11/4 2:35, Eduardo Valentin wrote:
> Hello,
>
> On Mon, Nov 03, 2014 at 06:17:37PM +0000, R, Durgadoss wrote:
>>> -----Original Message-----
>>> From: linux-pm-owner@vger.kernel.org [mailto:linux-pm-
>>> owner@vger.kernel.org] On Behalf Of Eduardo Valentin
>>> Sent: Monday, October 20, 2014 6:04 PM
>>> To: Yao Dongdong
>>> Cc: Zhang, Rui; linux-pm@vger.kernel.org; LKML
>>> Subject: Re: [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister
>>>
>>> Hello,
>>>
>>> On Mon, Oct 20, 2014 at 04:27:36PM +0800, Yao Dongdong wrote:
>>>> Signed-off-by:yaodongdong@huawei.com
>>> Acked-by: Eduardo Valentin <edubezval@gmail.com>
>>>
>>> Rui, would you take care of this?
>>>
>> If I remember it right, this 'tz' is freed in the thermal_release()
>> function, during device_unregister().
>>
>> It is similar in all other functions in thermal_core.c
>>
>> So, Yao, Did you really test this patch ?
>> And did not see any crashes ?
>>
> Yao, reading the patch change carefully, now I see that Durga is correct
> here.
>
>> Thanks,
>> Durga
>>
>>>> ---
>>>> drivers/thermal/thermal_core.c | 1 +
>>>> 1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
>>>> index 71b0ec0..5b7d466 100644
>>>> --- a/drivers/thermal/thermal_core.c
>>>> +++ b/drivers/thermal/thermal_core.c
>>>> @@ -1574,6 +1574,7 @@ struct thermal_zone_device
>>> *thermal_zone_device_register(const char *type,
>>>> unregister:
>>>> release_idr(&thermal_tz_idr, &thermal_idr_lock, tz->id);
>>>> device_unregister(&tz->device);
>>>> + kfree(tz);
> The device unregister is called above your kfree call, which will cause
> the thermal_release to be called. Did you test the code for double kfree
> calls? Your patch probably inserts a memory corruption.
>
> I will revert your patch from my tree until you provide an answer about
> your testing.
Yes, i have checked it and you are right, the patch will double kfree tz.
Thansks Durga for correcting me.
> Thanks,
>
>
> Eduardo Valentin
>
>>>> return ERR_PTR(result);
>>>> }
>>>> EXPORT_SYMBOL_GPL(thermal_zone_device_register);
>>>> --
>>>> 1.8.0.1
>>>>
>>>>
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-11-04 2:01 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-10-20 8:27 [PATCH 1/2] Thermal:Fix memory leak if occur goto unregister Yao Dongdong
2014-10-20 12:33 ` Eduardo Valentin
2014-11-03 18:17 ` R, Durgadoss
2014-11-03 18:35 ` Eduardo Valentin
2014-11-04 2:01 ` Yao Dongdong
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).