summaryrefslogtreecommitdiffstats
path: root/kernel/irq
diff options
context:
space:
mode:
authorDean Nelson <dcn@sgi.com>2008-10-18 16:06:56 -0700
committerIngo Molnar <mingo@elte.hu>2008-10-21 15:59:21 +0200
commitb6f3b7803a9231eddc36d0a2a6d2d8105ef89344 (patch)
treeac5149f718d8c6c0692daa973632f3538253c31c /kernel/irq
parente9f95e637320efe1936b647308ddf4ec5b8e0311 (diff)
downloadop-kernel-dev-b6f3b7803a9231eddc36d0a2a6d2d8105ef89344.zip
op-kernel-dev-b6f3b7803a9231eddc36d0a2a6d2d8105ef89344.tar.gz
genirq: NULL struct irq_desc's member 'name' in dynamic_irq_cleanup()
If the member 'name' of the irq_desc structure happens to point to a character string that is resident within a kernel module, problems ensue if that module is rmmod'd (at which time dynamic_irq_cleanup() is called) and then later show_interrupts() is called by someone. It is also not a good thing if the character string resided in kmalloc'd space that has been kfree'd (after having called dynamic_irq_cleanup()). dynamic_irq_cleanup() fails to NULL the 'name' member and show_interrupts() references it on a few architectures (like h8300, sh and x86). Signed-off-by: Dean Nelson <dcn@sgi.com> Signed-off-by: Ingo Molnar <mingo@elte.hu> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'kernel/irq')
-rw-r--r--kernel/irq/chip.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/kernel/irq/chip.c b/kernel/irq/chip.c
index 3de6ea3..10b5092 100644
--- a/kernel/irq/chip.c
+++ b/kernel/irq/chip.c
@@ -76,6 +76,7 @@ void dynamic_irq_cleanup(unsigned int irq)
desc->chip_data = NULL;
desc->handle_irq = handle_bad_irq;
desc->chip = &no_irq_chip;
+ desc->name = NULL;
spin_unlock_irqrestore(&desc->lock, flags);
}
OpenPOWER on IntegriCloud