diff options
| author | Chen Gang <gang.chen@asianux.com> | 2013-05-14 13:02:45 +0200 |
|---|---|---|
| committer | Grant Likely <grant.likely@linaro.org> | 2013-06-08 22:15:09 +0200 |
| commit | 275e31b10ce20613aedceaa5160129c64b260a98 (patch) | |
| tree | 07e207a672bc8adf3c732d3a1194c9bfce2bb69f | |
| parent | irqdomain: export irq_domain_add_simple (diff) | |
| download | linux-275e31b10ce20613aedceaa5160129c64b260a98.tar.xz linux-275e31b10ce20613aedceaa5160129c64b260a98.zip | |
kernel/irq/irqdomain.c: before use 'irq_data', need check it whether valid.
Since irq_data may be NULL, if so, we WARN_ON(), and continue, 'hwirq'
which related with 'irq_data' has to initialize later, or it will cause
issue.
Signed-off-by: Chen Gang <gang.chen@asianux.com>
Signed-off-by: Grant Likely <grant.likely@linaro.org>
| -rw-r--r-- | kernel/irq/irqdomain.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c index d1adaedb435f..8c4c8ea6a205 100644 --- a/kernel/irq/irqdomain.c +++ b/kernel/irq/irqdomain.c @@ -398,11 +398,12 @@ static void irq_domain_disassociate_many(struct irq_domain *domain, while (count--) { int irq = irq_base + count; struct irq_data *irq_data = irq_get_irq_data(irq); - irq_hw_number_t hwirq = irq_data->hwirq; + irq_hw_number_t hwirq; if (WARN_ON(!irq_data || irq_data->domain != domain)) continue; + hwirq = irq_data->hwirq; irq_set_status_flags(irq, IRQ_NOREQUEST); /* remove chip and handler */ |