diff options
author | Lior Dotan <liodot@gmail.com> | 2008-10-12 06:43:33 (GMT) |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-10-22 16:56:37 (GMT) |
commit | 786ed801ac9607a98fdeb8230ea05202a41fc1c7 (patch) | |
tree | 11b15f596c27f36bdc5409ecea4d7f87326eebd9 | |
parent | 9b2126db35bd7546c5998a06dcbac96c6ec21e4f (diff) | |
download | linux-fsl-qoriq-786ed801ac9607a98fdeb8230ea05202a41fc1c7.tar.xz |
Staging: SLICOSS: Free multicast list at driver exit
The multicast address list is allocated in slic_mcast_add_list() but
never freed.
Signed-off-by: Lior Dotan <liodot@gmail.com>
-rw-r--r-- | drivers/staging/slicoss/slicoss.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/staging/slicoss/slicoss.c b/drivers/staging/slicoss/slicoss.c index 6ef9274..8fa9490 100644 --- a/drivers/staging/slicoss/slicoss.c +++ b/drivers/staging/slicoss/slicoss.c @@ -596,6 +596,7 @@ static void __devexit slic_entry_remove(struct pci_dev *pcidev) uint mmio_len = 0; struct adapter *adapter = (struct adapter *) netdev_priv(dev); struct sliccard *card; + struct mcast_address *mcaddr, *mlist; ASSERT(adapter); DBG_MSG("slicoss: %s ENTER dev[%p] adapter[%p]\n", __func__, dev, @@ -615,6 +616,13 @@ static void __devexit slic_entry_remove(struct pci_dev *pcidev) DBG_MSG("slicoss: %s iounmap dev->base_addr[%x]\n", __func__, (uint) dev->base_addr); iounmap((void __iomem *)dev->base_addr); + /* free multicast addresses */ + mlist = adapter->mcastaddrs; + while (mlist) { + mcaddr = mlist; + mlist = mlist->next; + kfree(mcaddr); + } ASSERT(adapter->card); card = adapter->card; ASSERT(card->adapters_allocated); |