On Mon, Jan 5, 2015 at 1:04 AM, sanjeev sharma
<sanjeevsharmae...@gmail.com> wrote:
> On Fri, Jan 2, 2015 at 11:47 PM, Devendra Naga <devendra.a...@gmail.com>
> wrote:
>> the function cp_tm1217_mask_interrupt can return failure.
>> added the check and the failure path.
>>
>> Cc: Ramesh Agarwal <ramesh.agar...@intel.com>
>> Signed-off-by: Devendra Naga <devendra.a...@gmail.com>
>> ---
>>
>> build tested only on x86_64. config is allmodconfig.
>>
>> drivers/staging/cptm1217/clearpad_tm1217.c | 6 ++++++
>> 1 file changed, 6 insertions(+)
>>
>> diff --git a/drivers/staging/cptm1217/clearpad_tm1217.c
>> b/drivers/staging/cptm1217/clearpad_tm1217.c
>> index 7f265ce..54e5953 100644
>> --- a/drivers/staging/cptm1217/clearpad_tm1217.c
>> +++ b/drivers/staging/cptm1217/clearpad_tm1217.c
>> @@ -446,6 +446,12 @@ static int cp_tm1217_probe(struct i2c_client *client,
>>
>> /* Mask all the interrupts */
>> retval = cp_tm1217_mask_interrupt(ts);
>> + if (retval) {
>> + dev_err(ts->dev, "failed to mask interrupts, error: %d\n",
>> + retval);
>> + kfree(ts);
>
> Here you are doing more than what you have specified in change-log.How
> did you find out memory leak ? Did you used any facility like kmemleak
> facility to find this
> problem ?
No i am not doing more than one change. Read the change log one more time.
_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
