lkml.org 
[lkml]   [2020]   [Jun]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH 1/7] irqchip: Fix potential resource leaks
From
Date
On 06/23/2020 11:55 PM, Markus Elfring wrote:
>> There exists some potential resource leaks in the error path, fix them.
> Will the tag “Fixes” become relevant for the commit message?

Hi Markus,

Thanks for your reply and suggestion.

This patch contains many files in drivers/irqchip,
maybe I should split it into some small patches if use the tag "Fixes"?

>
>
> …
>> +++ b/drivers/irqchip/irq-nvic.c
>> @@ -94,6 +94,7 @@ static int __init nvic_of_init(struct device_node *node,
>>
>> if (!nvic_irq_domain) {
>> pr_warn("Failed to allocate irq domain\n");
>> + iounmap(nvic_base);
>> return -ENOMEM;
>> }
>>
>> @@ -103,6 +104,7 @@ static int __init nvic_of_init(struct device_node *node,
>> if (ret) {
>> pr_warn("Failed to allocate irq chips\n");
>> irq_domain_remove(nvic_irq_domain);
>> + iounmap(nvic_base);
>> return ret;
>> }
> Can it helpful to add jump targets so that a bit of exception handling
> can be better reused at the end of this function?

OK, no problem, I will do it in the v2.

By the way, I have resent this patch series due to git send-email failed,
https://lore.kernel.org/patchwork/cover/1261517/

Thanks,
Tiezhu

>
> Regards,
> Markus

\
 
 \ /
  Last update: 2020-06-24 03:46    [W:1.602 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site