Commit fa07ab72 authored Jun 11, 2017 by Heiner Kallweit Committed by Thomas Gleixner Jun 13, 2017

genirq: Release resources in __setup_irq() error path



In case __irq_set_trigger() fails the resources requested via
irq_request_resources() are not released.

Add the missing release call into the error handling path.

Fixes: c1bacbae ("genirq: Provide irq_request/release_resources chip callbacks")
Signed-off-by: Heiner Kallweit <hkallweit1@gmail.com>
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
Cc: stable@vger.kernel.org
Link: http://lkml.kernel.org/r/655538f5-cb20-a892-ff15-fbd2dd1fa4ec@gmail.com

parent 32c1431e

kernel/irq/manage.c

+3 −1

Original line number	Diff line number	Diff line
		@@ -1312,9 +1312,11 @@ __setup_irq(unsigned int irq, struct irq_desc desc, struct irqaction new)
		ret = __irq_set_trigger(desc,
		new->flags & IRQF_TRIGGER_MASK);

		if (ret)
		if (ret) {
		irq_release_resources(desc);
		goto out_mask;
		}
		}

		desc->istate &= ~(IRQS_AUTODETECT \| IRQS_SPURIOUS_DISABLED \| \
		IRQS_ONESHOT \| IRQS_WAITING);

Admin message