diff mbox

[v8,4/7] irqdomain: Check for NULL function pointer in irq_domain_free_irqs_hierarchy()

Message ID 1503017616-3252-5-git-send-email-david.daney@cavium.com
State New
Headers show

Commit Message

David Daney Aug. 18, 2017, 12:53 a.m. UTC 
A follow-on patch will call irq_domain_free_irqs_hierarchy() when the
free() function pointer may be NULL.

Add a NULL pointer check to handle this new use case.

Signed-off-by: David Daney <david.daney@cavium.com>
---
 kernel/irq/irqdomain.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Marc Zyngier Aug. 18, 2017, 9:16 a.m. UTC | #1 
On 18/08/17 01:53, David Daney wrote:
> A follow-on patch will call irq_domain_free_irqs_hierarchy() when the
> free() function pointer may be NULL.
> 
> Add a NULL pointer check to handle this new use case.
> 
> Signed-off-by: David Daney <david.daney@cavium.com>

Reviewed-by: Marc Zyngier <marc.zyngier@arm.com>

	M.
diff mbox

Patch

diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
index 2093b88..24fda75 100644
--- a/kernel/irq/irqdomain.c
+++ b/kernel/irq/irqdomain.c
@@ -1362,7 +1362,8 @@  static void irq_domain_free_irqs_hierarchy(struct irq_domain *domain,
 					   unsigned int irq_base,
 					   unsigned int nr_irqs)
 {
-	domain->ops->free(domain, irq_base, nr_irqs);
+	if (domain->ops->free)
+		domain->ops->free(domain, irq_base, nr_irqs);
 }
 
 int irq_domain_alloc_irqs_hierarchy(struct irq_domain *domain,