staging: slicoss: fix use-after-free in slic_entry_probe
authorDavid Matlack <dmatlack@google.com>
Tue, 6 May 2014 04:02:31 +0000 (21:02 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 23 May 2014 11:08:33 +0000 (20:08 +0900)
This patch fixes a use-after-free bug that can cause a kernel
oops. If slic_card_init fails then slic_entry_probe (the pci
probe() function for this device) will return error without
cleaning up memory.

Signed-off-by: David Matlack <dmatlack@google.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/slicoss/slicoss.c

index f350e6028ed2362a62ca480871a4b27e1da8aa8f..0b0a7b4e20e4265f25bffaf909351ee634c82290 100644 (file)
@@ -3594,7 +3594,6 @@ static int slic_entry_probe(struct pci_dev *pcidev,
        struct net_device *netdev;
        struct adapter *adapter;
        void __iomem *memmapped_ioaddr = NULL;
-       u32 status = 0;
        ulong mmio_start = 0;
        ulong mmio_len = 0;
        struct sliccard *card = NULL;
@@ -3685,16 +3684,11 @@ static int slic_entry_probe(struct pci_dev *pcidev,
                adapter->allocated = 1;
        }
 
-       status = slic_card_init(card, adapter);
+       err = slic_card_init(card, adapter);
+       if (err)
+               goto err_out_unmap;
 
-       if (status != 0) {
-               card->state = CARD_FAIL;
-               adapter->state = ADAPT_FAIL;
-               adapter->linkstate = LINK_DOWN;
-               dev_err(&pcidev->dev, "FAILED status[%x]\n", status);
-       } else {
-               slic_adapter_set_hwaddr(adapter);
-       }
+       slic_adapter_set_hwaddr(adapter);
 
        netdev->base_addr = (unsigned long)adapter->memorybase;
        netdev->irq = adapter->irq;
@@ -3711,7 +3705,7 @@ static int slic_entry_probe(struct pci_dev *pcidev,
 
        cards_found++;
 
-       return status;
+       return 0;
 
 err_out_unmap:
        iounmap(memmapped_ioaddr);