Fix reference leaks where PCI device references
obtained via pci_get_device() were not being released:

1. The while loop that iterates through 0xa00a devices was not
   releasing the final device reference when the loop terminates.

2. Single device lookups for 0xa001 and 0xa009 devices were not
   releasing their references after use.

Add missing pci_dev_put() calls to ensure all device references
are properly released.

Fixes: cd7834167ffb ("[POWERPC] pasemi: Print more information at machine 
check")
Cc: sta...@vger.kernel.org
Signed-off-by: Miaoqian Lin <linmq...@gmail.com>
---
 arch/powerpc/platforms/pasemi/setup.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/powerpc/platforms/pasemi/setup.c 
b/arch/powerpc/platforms/pasemi/setup.c
index d03b41336901..dafbee3afd86 100644
--- a/arch/powerpc/platforms/pasemi/setup.c
+++ b/arch/powerpc/platforms/pasemi/setup.c
@@ -169,6 +169,8 @@ static int __init pas_setup_mce_regs(void)
                dev = pci_get_device(PCI_VENDOR_ID_PASEMI, 0xa00a, dev);
                reg++;
        }
+       /* Release the last device reference from the while loop */
+       pci_dev_put(dev);
 
        dev = pci_get_device(PCI_VENDOR_ID_PASEMI, 0xa001, NULL);
        if (dev && reg+4 < MAX_MCE_REGS) {
@@ -185,6 +187,7 @@ static int __init pas_setup_mce_regs(void)
                mce_regs[reg].addr = pasemi_pci_getcfgaddr(dev, 0xc1c);
                reg++;
        }
+       pci_dev_put(dev);
 
        dev = pci_get_device(PCI_VENDOR_ID_PASEMI, 0xa009, NULL);
        if (dev && reg+2 < MAX_MCE_REGS) {
@@ -195,6 +198,7 @@ static int __init pas_setup_mce_regs(void)
                mce_regs[reg].addr = pasemi_pci_getcfgaddr(dev, 0x214);
                reg++;
        }
+       pci_dev_put(dev);
 
        num_mce_regs = reg;
 
-- 
2.35.1


Reply via email to