From 25e057fdde3b33c0d18605ed27b59336a3441925 Mon Sep 17 00:00:00 2001 From: Takeshi Yoshimura Date: Wed, 27 May 2015 11:28:14 -0300 Subject: [PATCH] [media] ddbridge: Do not free_irq() if request_irq() failed My static checker detected that free_irq() is called even after request_irq() failed in ddb_probe(). In this case, the kernel may try to free dev->pdev->irq although the IRQ is not assigned. This event rarely occurs, but always introduces a warning if it happens. "goto fail1" always results in disabling enabled MSI and releasing a requested IRQ. It seems like the former handling is necessary. So I added a conditional branch before the free_irq() (stat == 0 means request_irq() succeeds). Signed-off-by: Takeshi Yoshimura Signed-off-by: Mauro Carvalho Chehab --- Reading git-format-patch failed