2011-04-02 11:39:56

by Xiaotian Feng

[permalink] [raw]
Subject: [PATCH] genirq: fix cpumask leak in __setup_irq()

The allocated cpumask should be freed in __setup_irq().

Signed-off-by: Xiaotian Feng <[email protected]>
Cc: Thomas Gleixner <[email protected]>
---
kernel/irq/manage.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
index 12a80fd..07c1611 100644
--- a/kernel/irq/manage.c
+++ b/kernel/irq/manage.c
@@ -1051,6 +1051,7 @@ __setup_irq(unsigned int irq, struct irq_desc *desc, struct irqaction *new)
register_irq_proc(irq, desc);
new->dir = NULL;
register_handler_proc(irq, new);
+ free_cpumask_var(mask);

return 0;

--
1.7.1


2011-04-02 19:31:09

by Xiaotian Feng

[permalink] [raw]
Subject: [tip:irq/urgent] genirq: Fix cpumask leak in __setup_irq()

Commit-ID: 4f5058c3b71ed5930bb2b478c4d5dbc799dd9ad1
Gitweb: http://git.kernel.org/tip/4f5058c3b71ed5930bb2b478c4d5dbc799dd9ad1
Author: Xiaotian Feng <[email protected]>
AuthorDate: Sat, 2 Apr 2011 19:39:35 +0800
Committer: Thomas Gleixner <[email protected]>
CommitDate: Sat, 2 Apr 2011 21:26:20 +0200

genirq: Fix cpumask leak in __setup_irq()

The allocated cpumask should be freed in __setup_irq().

Signed-off-by: Xiaotian Feng <[email protected]>
LKML-Reference: <[email protected]>
Signed-off-by: Thomas Gleixner <[email protected]>
---
kernel/irq/manage.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/kernel/irq/manage.c b/kernel/irq/manage.c
index 12a80fd..07c1611 100644
--- a/kernel/irq/manage.c
+++ b/kernel/irq/manage.c
@@ -1051,6 +1051,7 @@ __setup_irq(unsigned int irq, struct irq_desc *desc, struct irqaction *new)
register_irq_proc(irq, desc);
new->dir = NULL;
register_handler_proc(irq, new);
+ free_cpumask_var(mask);

return 0;