diff options
author | Jesper Juhl <jesper.juhl@gmail.com> | 2005-10-28 16:53:13 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@pobox.com> | 2005-10-28 16:53:13 -0400 |
commit | b4558ea93d66a43f7990d26f145fd4c54a01c9bf (patch) | |
tree | 70aa8ba4864f8ee994b7f5278f5045af6a646d34 /drivers/net | |
parent | 7380a78a973a8109c13cb0e47617c456b6f6e1f5 (diff) |
drivers/net: Remove pointless checks for NULL prior to calling kfree()
Diffstat (limited to 'drivers/net')
30 files changed, 88 insertions, 170 deletions
diff --git a/drivers/net/acenic.c b/drivers/net/acenic.c index dbecc6bf785..b8953de5664 100644 --- a/drivers/net/acenic.c +++ b/drivers/net/acenic.c | |||
@@ -871,10 +871,8 @@ static void ace_init_cleanup(struct net_device *dev) | |||
871 | if (ap->info) | 871 | if (ap->info) |
872 | pci_free_consistent(ap->pdev, sizeof(struct ace_info), | 872 | pci_free_consistent(ap->pdev, sizeof(struct ace_info), |
873 | ap->info, ap->info_dma); | 873 | ap->info, ap->info_dma); |
874 | if (ap->skb) | 874 | kfree(ap->skb); |
875 | kfree(ap->skb); | 875 | kfree(ap->trace_buf); |
876 | if (ap->trace_buf) | ||
877 | kfree(ap->trace_buf); | ||
878 | 876 | ||
879 | if (dev->irq) | 877 | if (dev->irq) |
880 | free_irq(dev->irq, dev); | 878 | free_irq(dev->irq, dev); |
diff --git a/drivers/net/au1000_eth.c b/drivers/net/au1000_eth.c index 78506911d65..332e9953c55 100644 --- a/drivers/net/au1000_eth.c +++ b/drivers/net/au1000_eth.c | |||
@@ -1606,8 +1606,7 @@ err_out: | |||
1606 | /* here we should have a valid dev plus aup-> register addresses | 1606 | /* here we should have a valid dev plus aup-> register addresses |
1607 | * so we can reset the mac properly.*/ | 1607 | * so we can reset the mac properly.*/ |
1608 | reset_mac(dev); | 1608 | reset_mac(dev); |
1609 | if (aup->mii) | 1609 | kfree(aup->mii); |
1610 | kfree(aup->mii); | ||
1611 | for (i = 0; i < NUM_RX_DMA; i++) { | 1610 | for (i = 0; i < NUM_RX_DMA; i++) { |
1612 | if (aup->rx_db_inuse[i]) | 1611 | if (aup->rx_db_inuse[i]) |
1613 | ReleaseDB(aup, aup->rx_db_inuse[i]); | 1612 | ReleaseDB(aup, aup->rx_db_inuse[i]); |
@@ -1806,8 +1805,7 @@ static void __exit au1000_cleanup_module(void) | |||
1806 | if (dev) { | 1805 | if (dev) { |
1807 | aup = (struct au1000_private *) dev->priv; | 1806 | aup = (struct au1000_private *) dev->priv; |
1808 | unregister_netdev(dev); | 1807 | unregister_netdev(dev); |
1809 | if (aup->mii) | 1808 | kfree(aup->mii); |
1810 | kfree(aup->mii); | ||
1811 | for (j = 0; j < NUM_RX_DMA; j++) { | 1809 | for (j = 0; j < NUM_RX_DMA; j++) { |
1812 | if (aup->rx_db_inuse[j]) | 1810 | if (aup->rx_db_inuse[j]) |
1813 | ReleaseDB(aup, aup->rx_db_inuse[j]); | 1811 | ReleaseDB(aup, aup->rx_db_inuse[j]); |
diff --git a/drivers/net/b44.c b/drivers/net/b44.c index ab845076ff9..5485e3b1cd3 100644 --- a/drivers/net/b44.c +++ b/drivers/net/b44.c | |||
@@ -1131,14 +1131,10 @@ static void b44_init_rings(struct b44 *bp) | |||
1131 | */ | 1131 | */ |
1132 | static void b44_free_consistent(struct b44 *bp) | 1132 | static void b44_free_consistent(struct b44 *bp) |
1133 | { | 1133 | { |
1134 | if (bp->rx_buffers) { | 1134 | kfree(bp->rx_buffers); |
1135 | kfree(bp->rx_buffers); | 1135 | bp->rx_buffers = NULL; |
1136 | bp->rx_buffers = NULL; | 1136 | kfree(bp->tx_buffers); |
1137 | } | 1137 | bp->tx_buffers = NULL; |
1138 | if (bp->tx_buffers) { | ||
1139 | kfree(bp->tx_buffers); | ||
1140 | bp->tx_buffers = NULL; | ||
1141 | } | ||
1142 | if (bp->rx_ring) { | 1138 | if (bp->rx_ring) { |
1143 | if (bp->flags & B44_FLAG_RX_RING_HACK) { | 1139 | if (bp->flags & B44_FLAG_RX_RING_HACK) { |
1144 | dma_unmap_single(&bp->pdev->dev, bp->rx_ring_dma, | 1140 | dma_unmap_single(&bp->pdev->dev, bp->rx_ring_dma, |
diff --git a/drivers/net/bmac.c b/drivers/net/bmac.c index 60dba4a1ca5..73f2fcfc557 100644 --- a/drivers/net/bmac.c +++ b/drivers/net/bmac.c | |||
@@ -1689,10 +1689,8 @@ static void __exit bmac_exit(void) | |||
1689 | { | 1689 | { |
1690 | macio_unregister_driver(&bmac_driver); | 1690 | macio_unregister_driver(&bmac_driver); |
1691 | 1691 | ||
1692 | if (bmac_emergency_rxbuf != NULL) { | 1692 | kfree(bmac_emergency_rxbuf); |
1693 | kfree(bmac_emergency_rxbuf); | 1693 | bmac_emergency_rxbuf = NULL; |
1694 | bmac_emergency_rxbuf = NULL; | ||
1695 | } | ||
1696 | } | 1694 | } |
1697 | 1695 | ||
1698 | MODULE_AUTHOR("Randy Gobbel/Paul Mackerras"); | 1696 | MODULE_AUTHOR("Randy Gobbel/Paul Mackerras"); |
diff --git a/drivers/net/bnx2.c b/drivers/net/bnx2.c index 3a2ace01e44..11d25231822 100644 --- a/drivers/net/bnx2.c +++ b/drivers/net/bnx2.c | |||
@@ -314,20 +314,16 @@ bnx2_free_mem(struct bnx2 *bp) | |||
314 | bp->tx_desc_ring, bp->tx_desc_mapping); | 314 | bp->tx_desc_ring, bp->tx_desc_mapping); |
315 | bp->tx_desc_ring = NULL; | 315 | bp->tx_desc_ring = NULL; |
316 | } | 316 | } |
317 | if (bp->tx_buf_ring) { | 317 | kfree(bp->tx_buf_ring); |
318 | kfree(bp->tx_buf_ring); | 318 | bp->tx_buf_ring = NULL; |
319 | bp->tx_buf_ring = NULL; | ||
320 | } | ||
321 | if (bp->rx_desc_ring) { | 319 | if (bp->rx_desc_ring) { |
322 | pci_free_consistent(bp->pdev, | 320 | pci_free_consistent(bp->pdev, |
323 | sizeof(struct rx_bd) * RX_DESC_CNT, | 321 | sizeof(struct rx_bd) * RX_DESC_CNT, |
324 | bp->rx_desc_ring, bp->rx_desc_mapping); | 322 | bp->rx_desc_ring, bp->rx_desc_mapping); |
325 | bp->rx_desc_ring = NULL; | 323 | bp->rx_desc_ring = NULL; |
326 | } | 324 | } |
327 | if (bp->rx_buf_ring) { | 325 | kfree(bp->rx_buf_ring); |
328 | kfree(bp->rx_buf_ring); | 326 | bp->rx_buf_ring = NULL; |
329 | bp->rx_buf_ring = NULL; | ||
330 | } | ||
331 | } | 327 | } |
332 | 328 | ||
333 | static int | 329 | static int |
diff --git a/drivers/net/e1000/e1000_ethtool.c b/drivers/net/e1000/e1000_ethtool.c index 6b9acc7f94a..9c7feaeaa6a 100644 --- a/drivers/net/e1000/e1000_ethtool.c +++ b/drivers/net/e1000/e1000_ethtool.c | |||
@@ -965,11 +965,8 @@ e1000_free_desc_rings(struct e1000_adapter *adapter) | |||
965 | if(rxdr->desc) | 965 | if(rxdr->desc) |
966 | pci_free_consistent(pdev, rxdr->size, rxdr->desc, rxdr->dma); | 966 | pci_free_consistent(pdev, rxdr->size, rxdr->desc, rxdr->dma); |
967 | 967 | ||
968 | if(txdr->buffer_info) | 968 | kfree(txdr->buffer_info); |
969 | kfree(txdr->buffer_info); | 969 | kfree(rxdr->buffer_info); |
970 | if(rxdr->buffer_info) | ||
971 | kfree(rxdr->buffer_info); | ||
972 | |||
973 | return; | 970 | return; |
974 | } | 971 | } |
975 | 972 | ||
diff --git a/drivers/net/hamradio/mkiss.c b/drivers/net/hamradio/mkiss.c index 85d6dc005be..3e9accf137e 100644 --- a/drivers/net/hamradio/mkiss.c +++ b/drivers/net/hamradio/mkiss.c | |||
@@ -390,10 +390,8 @@ static void ax_changedmtu(struct mkiss *ax) | |||
390 | "MTU change cancelled.\n", | 390 | "MTU change cancelled.\n", |
391 | ax->dev->name); | 391 | ax->dev->name); |
392 | dev->mtu = ax->mtu; | 392 | dev->mtu = ax->mtu; |
393 | if (xbuff != NULL) | 393 | kfree(xbuff); |
394 | kfree(xbuff); | 394 | kfree(rbuff); |
395 | if (rbuff != NULL) | ||
396 | kfree(rbuff); | ||
397 | return; | 395 | return; |
398 | } | 396 | } |
399 | 397 | ||
diff --git a/drivers/net/ibmveth.c b/drivers/net/ibmveth.c index f5819527ec9..36da54ad2b7 100644 --- a/drivers/net/ibmveth.c +++ b/drivers/net/ibmveth.c | |||
@@ -276,10 +276,8 @@ static void ibmveth_free_buffer_pool(struct ibmveth_adapter *adapter, struct ibm | |||
276 | { | 276 | { |
277 | int i; | 277 | int i; |
278 | 278 | ||
279 | if(pool->free_map) { | 279 | kfree(pool->free_map); |
280 | kfree(pool->free_map); | 280 | pool->free_map = NULL; |
281 | pool->free_map = NULL; | ||
282 | } | ||
283 | 281 | ||
284 | if(pool->skbuff && pool->dma_addr) { | 282 | if(pool->skbuff && pool->dma_addr) { |
285 | for(i = 0; i < pool->size; ++i) { | 283 | for(i = 0; i < pool->size; ++i) { |
diff --git a/drivers/net/irda/donauboe.c b/drivers/net/irda/donauboe.c index 0a08c539c05..0282771b1cb 100644 --- a/drivers/net/irda/donauboe.c +++ b/drivers/net/irda/donauboe.c | |||
@@ -1695,11 +1695,9 @@ toshoboe_open (struct pci_dev *pci_dev, const struct pci_device_id *pdid) | |||
1695 | 1695 | ||
1696 | freebufs: | 1696 | freebufs: |
1697 | for (i = 0; i < TX_SLOTS; ++i) | 1697 | for (i = 0; i < TX_SLOTS; ++i) |
1698 | if (self->tx_bufs[i]) | 1698 | kfree (self->tx_bufs[i]); |
1699 | kfree (self->tx_bufs[i]); | ||
1700 | for (i = 0; i < RX_SLOTS; ++i) | 1699 | for (i = 0; i < RX_SLOTS; ++i) |
1701 | if (self->rx_bufs[i]) | 1700 | kfree (self->rx_bufs[i]); |
1702 | kfree (self->rx_bufs[i]); | ||
1703 | kfree(self->ringbuf); | 1701 | kfree(self->ringbuf); |
1704 | 1702 | ||
1705 | freeregion: | 1703 | freeregion: |
diff --git a/drivers/net/irda/irda-usb.c b/drivers/net/irda/irda-usb.c index 6c766fdc51a..c22c0517883 100644 --- a/drivers/net/irda/irda-usb.c +++ b/drivers/net/irda/irda-usb.c | |||
@@ -1168,10 +1168,8 @@ static inline void irda_usb_close(struct irda_usb_cb *self) | |||
1168 | unregister_netdev(self->netdev); | 1168 | unregister_netdev(self->netdev); |
1169 | 1169 | ||
1170 | /* Remove the speed buffer */ | 1170 | /* Remove the speed buffer */ |
1171 | if (self->speed_buff != NULL) { | 1171 | kfree(self->speed_buff); |
1172 | kfree(self->speed_buff); | 1172 | self->speed_buff = NULL; |
1173 | self->speed_buff = NULL; | ||
1174 | } | ||
1175 | } | 1173 | } |
1176 | 1174 | ||
1177 | /********************** USB CONFIG SUBROUTINES **********************/ | 1175 | /********************** USB CONFIG SUBROUTINES **********************/ |
diff --git a/drivers/net/irda/irport.c b/drivers/net/irda/irport.c index 5971315f3fa..3d016a498e1 100644 --- a/drivers/net/irda/irport.c +++ b/drivers/net/irda/irport.c | |||
@@ -235,8 +235,7 @@ static int irport_close(struct irport_cb *self) | |||
235 | __FUNCTION__, self->io.sir_base); | 235 | __FUNCTION__, self->io.sir_base); |
236 | release_region(self->io.sir_base, self->io.sir_ext); | 236 | release_region(self->io.sir_base, self->io.sir_ext); |
237 | 237 | ||
238 | if (self->tx_buff.head) | 238 | kfree(self->tx_buff.head); |
239 | kfree(self->tx_buff.head); | ||
240 | 239 | ||
241 | if (self->rx_buff.skb) | 240 | if (self->rx_buff.skb) |
242 | kfree_skb(self->rx_buff.skb); | 241 | kfree_skb(self->rx_buff.skb); |
diff --git a/drivers/net/irda/sir_dev.c b/drivers/net/irda/sir_dev.c index efc5a887056..df22b8b532e 100644 --- a/drivers/net/irda/sir_dev.c +++ b/drivers/net/irda/sir_dev.c | |||
@@ -490,8 +490,7 @@ static void sirdev_free_buffers(struct sir_dev *dev) | |||
490 | { | 490 | { |
491 | if (dev->rx_buff.skb) | 491 | if (dev->rx_buff.skb) |
492 | kfree_skb(dev->rx_buff.skb); | 492 | kfree_skb(dev->rx_buff.skb); |
493 | if (dev->tx_buff.head) | 493 | kfree(dev->tx_buff.head); |
494 | kfree(dev->tx_buff.head); | ||
495 | dev->rx_buff.head = dev->tx_buff.head = NULL; | 494 | dev->rx_buff.head = dev->tx_buff.head = NULL; |
496 | dev->rx_buff.skb = NULL; | 495 | dev->rx_buff.skb = NULL; |
497 | } | 496 | } |
diff --git a/drivers/net/irda/vlsi_ir.c b/drivers/net/irda/vlsi_ir.c index 651c5a6578f..a9f49f058cf 100644 --- a/drivers/net/irda/vlsi_ir.c +++ b/drivers/net/irda/vlsi_ir.c | |||
@@ -473,8 +473,7 @@ static int vlsi_free_ring(struct vlsi_ring *r) | |||
473 | rd_set_addr_status(rd, 0, 0); | 473 | rd_set_addr_status(rd, 0, 0); |
474 | if (busaddr) | 474 | if (busaddr) |
475 | pci_unmap_single(r->pdev, busaddr, r->len, r->dir); | 475 | pci_unmap_single(r->pdev, busaddr, r->len, r->dir); |
476 | if (rd->buf) | 476 | kfree(rd->buf); |
477 | kfree(rd->buf); | ||
478 | } | 477 | } |
479 | kfree(r); | 478 | kfree(r); |
480 | return 0; | 479 | return 0; |
diff --git a/drivers/net/mace.c b/drivers/net/mace.c index 81d0a26e4f4..09b1e7b364e 100644 --- a/drivers/net/mace.c +++ b/drivers/net/mace.c | |||
@@ -1035,10 +1035,8 @@ static void __exit mace_cleanup(void) | |||
1035 | { | 1035 | { |
1036 | macio_unregister_driver(&mace_driver); | 1036 | macio_unregister_driver(&mace_driver); |
1037 | 1037 | ||
1038 | if (dummy_buf) { | 1038 | kfree(dummy_buf); |
1039 | kfree(dummy_buf); | 1039 | dummy_buf = NULL; |
1040 | dummy_buf = NULL; | ||
1041 | } | ||
1042 | } | 1040 | } |
1043 | 1041 | ||
1044 | MODULE_AUTHOR("Paul Mackerras"); | 1042 | MODULE_AUTHOR("Paul Mackerras"); |
diff --git a/drivers/net/ni65.c b/drivers/net/ni65.c index 925d1dfcc4d..bb42ff21848 100644 --- a/drivers/net/ni65.c +++ b/drivers/net/ni65.c | |||
@@ -696,8 +696,7 @@ static void ni65_free_buffer(struct priv *p) | |||
696 | return; | 696 | return; |
697 | 697 | ||
698 | for(i=0;i<TMDNUM;i++) { | 698 | for(i=0;i<TMDNUM;i++) { |
699 | if(p->tmdbounce[i]) | 699 | kfree(p->tmdbounce[i]); |
700 | kfree(p->tmdbounce[i]); | ||
701 | #ifdef XMT_VIA_SKB | 700 | #ifdef XMT_VIA_SKB |
702 | if(p->tmd_skb[i]) | 701 | if(p->tmd_skb[i]) |
703 | dev_kfree_skb(p->tmd_skb[i]); | 702 | dev_kfree_skb(p->tmd_skb[i]); |
@@ -710,12 +709,10 @@ static void ni65_free_buffer(struct priv *p) | |||
710 | if(p->recv_skb[i]) | 709 | if(p->recv_skb[i]) |
711 | dev_kfree_skb(p->recv_skb[i]); | 710 | dev_kfree_skb(p->recv_skb[i]); |
712 | #else | 711 | #else |
713 | if(p->recvbounce[i]) | 712 | kfree(p->recvbounce[i]); |
714 | kfree(p->recvbounce[i]); | ||
715 | #endif | 713 | #endif |
716 | } | 714 | } |
717 | if(p->self) | 715 | kfree(p->self); |
718 | kfree(p->self); | ||
719 | } | 716 | } |
720 | 717 | ||
721 | 718 | ||
diff --git a/drivers/net/rrunner.c b/drivers/net/rrunner.c index ec1a18d189a..19c2df9c86f 100644 --- a/drivers/net/rrunner.c +++ b/drivers/net/rrunner.c | |||
@@ -1710,10 +1710,8 @@ static int rr_ioctl(struct net_device *dev, struct ifreq *rq, int cmd) | |||
1710 | error = -EFAULT; | 1710 | error = -EFAULT; |
1711 | } | 1711 | } |
1712 | wf_out: | 1712 | wf_out: |
1713 | if (oldimage) | 1713 | kfree(oldimage); |
1714 | kfree(oldimage); | 1714 | kfree(image); |
1715 | if (image) | ||
1716 | kfree(image); | ||
1717 | return error; | 1715 | return error; |
1718 | 1716 | ||
1719 | case SIOCRRID: | 1717 | case SIOCRRID: |
diff --git a/drivers/net/s2io.c b/drivers/net/s2io.c index d303d162974..5ea897714a1 100644 --- a/drivers/net/s2io.c +++ b/drivers/net/s2io.c | |||
@@ -705,8 +705,7 @@ static void free_shared_mem(struct s2io_nic *nic) | |||
705 | } | 705 | } |
706 | kfree(mac_control->rings[i].ba[j]); | 706 | kfree(mac_control->rings[i].ba[j]); |
707 | } | 707 | } |
708 | if (mac_control->rings[i].ba) | 708 | kfree(mac_control->rings[i].ba); |
709 | kfree(mac_control->rings[i].ba); | ||
710 | } | 709 | } |
711 | #endif | 710 | #endif |
712 | 711 | ||
diff --git a/drivers/net/saa9730.c b/drivers/net/saa9730.c index fd0167077fb..110e777f206 100644 --- a/drivers/net/saa9730.c +++ b/drivers/net/saa9730.c | |||
@@ -997,10 +997,7 @@ static void __devexit saa9730_remove_one(struct pci_dev *pdev) | |||
997 | 997 | ||
998 | if (dev) { | 998 | if (dev) { |
999 | unregister_netdev(dev); | 999 | unregister_netdev(dev); |
1000 | 1000 | kfree(dev->priv); | |
1001 | if (dev->priv) | ||
1002 | kfree(dev->priv); | ||
1003 | |||
1004 | free_netdev(dev); | 1001 | free_netdev(dev); |
1005 | pci_release_regions(pdev); | 1002 | pci_release_regions(pdev); |
1006 | pci_disable_device(pdev); | 1003 | pci_disable_device(pdev); |
@@ -1096,8 +1093,7 @@ static int lan_saa9730_init(struct net_device *dev, int ioaddr, int irq) | |||
1096 | return 0; | 1093 | return 0; |
1097 | 1094 | ||
1098 | out: | 1095 | out: |
1099 | if (dev->priv) | 1096 | kfree(dev->priv); |
1100 | kfree(dev->priv); | ||
1101 | return ret; | 1097 | return ret; |
1102 | } | 1098 | } |
1103 | 1099 | ||
diff --git a/drivers/net/tg3.c b/drivers/net/tg3.c index 551c9449322..b547233ad9f 100644 --- a/drivers/net/tg3.c +++ b/drivers/net/tg3.c | |||
@@ -3918,10 +3918,8 @@ static void tg3_init_rings(struct tg3 *tp) | |||
3918 | */ | 3918 | */ |
3919 | static void tg3_free_consistent(struct tg3 *tp) | 3919 | static void tg3_free_consistent(struct tg3 *tp) |
3920 | { | 3920 | { |
3921 | if (tp->rx_std_buffers) { | 3921 | kfree(tp->rx_std_buffers); |
3922 | kfree(tp->rx_std_buffers); | 3922 | tp->rx_std_buffers = NULL; |
3923 | tp->rx_std_buffers = NULL; | ||
3924 | } | ||
3925 | if (tp->rx_std) { | 3923 | if (tp->rx_std) { |
3926 | pci_free_consistent(tp->pdev, TG3_RX_RING_BYTES, | 3924 | pci_free_consistent(tp->pdev, TG3_RX_RING_BYTES, |
3927 | tp->rx_std, tp->rx_std_mapping); | 3925 | tp->rx_std, tp->rx_std_mapping); |
diff --git a/drivers/net/tulip/de2104x.c b/drivers/net/tulip/de2104x.c index 6b8eee8f7bf..d7fb3ffe06a 100644 --- a/drivers/net/tulip/de2104x.c +++ b/drivers/net/tulip/de2104x.c | |||
@@ -2076,8 +2076,7 @@ static int __init de_init_one (struct pci_dev *pdev, | |||
2076 | return 0; | 2076 | return 0; |
2077 | 2077 | ||
2078 | err_out_iomap: | 2078 | err_out_iomap: |
2079 | if (de->ee_data) | 2079 | kfree(de->ee_data); |
2080 | kfree(de->ee_data); | ||
2081 | iounmap(regs); | 2080 | iounmap(regs); |
2082 | err_out_res: | 2081 | err_out_res: |
2083 | pci_release_regions(pdev); | 2082 | pci_release_regions(pdev); |
@@ -2096,8 +2095,7 @@ static void __exit de_remove_one (struct pci_dev *pdev) | |||
2096 | if (!dev) | 2095 | if (!dev) |
2097 | BUG(); | 2096 | BUG(); |
2098 | unregister_netdev(dev); | 2097 | unregister_netdev(dev); |
2099 | if (de->ee_data) | 2098 | kfree(de->ee_data); |
2100 | kfree(de->ee_data); | ||
2101 | iounmap(de->regs); | 2099 | iounmap(de->regs); |
2102 | pci_release_regions(pdev); | 2100 | pci_release_regions(pdev); |
2103 | pci_disable_device(pdev); | 2101 | pci_disable_device(pdev); |
diff --git a/drivers/net/tulip/tulip_core.c b/drivers/net/tulip/tulip_core.c index 6266a9a7e6e..125ed00e95a 100644 --- a/drivers/net/tulip/tulip_core.c +++ b/drivers/net/tulip/tulip_core.c | |||
@@ -1727,8 +1727,7 @@ err_out_free_ring: | |||
1727 | tp->rx_ring, tp->rx_ring_dma); | 1727 | tp->rx_ring, tp->rx_ring_dma); |
1728 | 1728 | ||
1729 | err_out_mtable: | 1729 | err_out_mtable: |
1730 | if (tp->mtable) | 1730 | kfree (tp->mtable); |
1731 | kfree (tp->mtable); | ||
1732 | pci_iounmap(pdev, ioaddr); | 1731 | pci_iounmap(pdev, ioaddr); |
1733 | 1732 | ||
1734 | err_out_free_res: | 1733 | err_out_free_res: |
@@ -1806,8 +1805,7 @@ static void __devexit tulip_remove_one (struct pci_dev *pdev) | |||
1806 | sizeof (struct tulip_rx_desc) * RX_RING_SIZE + | 1805 | sizeof (struct tulip_rx_desc) * RX_RING_SIZE + |
1807 | sizeof (struct tulip_tx_desc) * TX_RING_SIZE, | 1806 | sizeof (struct tulip_tx_desc) * TX_RING_SIZE, |
1808 | tp->rx_ring, tp->rx_ring_dma); | 1807 | tp->rx_ring, tp->rx_ring_dma); |
1809 | if (tp->mtable) | 1808 | kfree (tp->mtable); |
1810 | kfree (tp->mtable); | ||
1811 | pci_iounmap(pdev, tp->base_addr); | 1809 | pci_iounmap(pdev, tp->base_addr); |
1812 | free_netdev (dev); | 1810 | free_netdev (dev); |
1813 | pci_release_regions (pdev); | 1811 | pci_release_regions (pdev); |
diff --git a/drivers/net/via-velocity.c b/drivers/net/via-velocity.c index abc5cee6eed..a368d08e7d1 100644 --- a/drivers/net/via-velocity.c +++ b/drivers/net/via-velocity.c | |||
@@ -1212,10 +1212,8 @@ static void velocity_free_td_ring(struct velocity_info *vptr) | |||
1212 | velocity_free_td_ring_entry(vptr, j, i); | 1212 | velocity_free_td_ring_entry(vptr, j, i); |
1213 | 1213 | ||
1214 | } | 1214 | } |
1215 | if (vptr->td_infos[j]) { | 1215 | kfree(vptr->td_infos[j]); |
1216 | kfree(vptr->td_infos[j]); | 1216 | vptr->td_infos[j] = NULL; |
1217 | vptr->td_infos[j] = NULL; | ||
1218 | } | ||
1219 | } | 1217 | } |
1220 | } | 1218 | } |
1221 | 1219 | ||
diff --git a/drivers/net/wireless/airo.c b/drivers/net/wireless/airo.c index cb429e78374..4c11699bad9 100644 --- a/drivers/net/wireless/airo.c +++ b/drivers/net/wireless/airo.c | |||
@@ -2381,14 +2381,10 @@ void stop_airo_card( struct net_device *dev, int freeres ) | |||
2381 | dev_kfree_skb(skb); | 2381 | dev_kfree_skb(skb); |
2382 | } | 2382 | } |
2383 | 2383 | ||
2384 | if (ai->flash) | 2384 | kfree(ai->flash); |
2385 | kfree(ai->flash); | 2385 | kfree(ai->rssi); |
2386 | if (ai->rssi) | 2386 | kfree(ai->APList); |
2387 | kfree(ai->rssi); | 2387 | kfree(ai->SSID); |
2388 | if (ai->APList) | ||
2389 | kfree(ai->APList); | ||
2390 | if (ai->SSID) | ||
2391 | kfree(ai->SSID); | ||
2392 | if (freeres) { | 2388 | if (freeres) { |
2393 | /* PCMCIA frees this stuff, so only for PCI and ISA */ | 2389 | /* PCMCIA frees this stuff, so only for PCI and ISA */ |
2394 | release_region( dev->base_addr, 64 ); | 2390 | release_region( dev->base_addr, 64 ); |
@@ -3626,10 +3622,8 @@ static u16 setup_card(struct airo_info *ai, u8 *mac, int lock) | |||
3626 | int rc; | 3622 | int rc; |
3627 | 3623 | ||
3628 | memset( &mySsid, 0, sizeof( mySsid ) ); | 3624 | memset( &mySsid, 0, sizeof( mySsid ) ); |
3629 | if (ai->flash) { | 3625 | kfree (ai->flash); |
3630 | kfree (ai->flash); | 3626 | ai->flash = NULL; |
3631 | ai->flash = NULL; | ||
3632 | } | ||
3633 | 3627 | ||
3634 | /* The NOP is the first step in getting the card going */ | 3628 | /* The NOP is the first step in getting the card going */ |
3635 | cmd.cmd = NOP; | 3629 | cmd.cmd = NOP; |
@@ -3666,14 +3660,10 @@ static u16 setup_card(struct airo_info *ai, u8 *mac, int lock) | |||
3666 | tdsRssiRid rssi_rid; | 3660 | tdsRssiRid rssi_rid; |
3667 | CapabilityRid cap_rid; | 3661 | CapabilityRid cap_rid; |
3668 | 3662 | ||
3669 | if (ai->APList) { | 3663 | kfree(ai->APList); |
3670 | kfree(ai->APList); | 3664 | ai->APList = NULL; |
3671 | ai->APList = NULL; | 3665 | kfree(ai->SSID); |
3672 | } | 3666 | ai->SSID = NULL; |
3673 | if (ai->SSID) { | ||
3674 | kfree(ai->SSID); | ||
3675 | ai->SSID = NULL; | ||
3676 | } | ||
3677 | // general configuration (read/modify/write) | 3667 | // general configuration (read/modify/write) |
3678 | status = readConfigRid(ai, lock); | 3668 | status = readConfigRid(ai, lock); |
3679 | if ( status != SUCCESS ) return ERROR; | 3669 | if ( status != SUCCESS ) return ERROR; |
@@ -3687,10 +3677,8 @@ static u16 setup_card(struct airo_info *ai, u8 *mac, int lock) | |||
3687 | memcpy(ai->rssi, (u8*)&rssi_rid + 2, 512); /* Skip RID length member */ | 3677 | memcpy(ai->rssi, (u8*)&rssi_rid + 2, 512); /* Skip RID length member */ |
3688 | } | 3678 | } |
3689 | else { | 3679 | else { |
3690 | if (ai->rssi) { | 3680 | kfree(ai->rssi); |
3691 | kfree(ai->rssi); | 3681 | ai->rssi = NULL; |
3692 | ai->rssi = NULL; | ||
3693 | } | ||
3694 | if (cap_rid.softCap & 8) | 3682 | if (cap_rid.softCap & 8) |
3695 | ai->config.rmode |= RXMODE_NORMALIZED_RSSI; | 3683 | ai->config.rmode |= RXMODE_NORMALIZED_RSSI; |
3696 | else | 3684 | else |
@@ -5369,11 +5357,13 @@ static int proc_BSSList_open( struct inode *inode, struct file *file ) { | |||
5369 | 5357 | ||
5370 | static int proc_close( struct inode *inode, struct file *file ) | 5358 | static int proc_close( struct inode *inode, struct file *file ) |
5371 | { | 5359 | { |
5372 | struct proc_data *data = (struct proc_data *)file->private_data; | 5360 | struct proc_data *data = file->private_data; |
5373 | if ( data->on_close != NULL ) data->on_close( inode, file ); | 5361 | |
5374 | if ( data->rbuffer ) kfree( data->rbuffer ); | 5362 | if (data->on_close != NULL) |
5375 | if ( data->wbuffer ) kfree( data->wbuffer ); | 5363 | data->on_close(inode, file); |
5376 | kfree( data ); | 5364 | kfree(data->rbuffer); |
5365 | kfree(data->wbuffer); | ||
5366 | kfree(data); | ||
5377 | return 0; | 5367 | return 0; |
5378 | } | 5368 | } |
5379 | 5369 | ||
diff --git a/drivers/net/wireless/airo_cs.c b/drivers/net/wireless/airo_cs.c index bf25584d68d..784de910911 100644 --- a/drivers/net/wireless/airo_cs.c +++ b/drivers/net/wireless/airo_cs.c | |||
@@ -258,9 +258,7 @@ static void airo_detach(dev_link_t *link) | |||
258 | 258 | ||
259 | /* Unlink device structure, free pieces */ | 259 | /* Unlink device structure, free pieces */ |
260 | *linkp = link->next; | 260 | *linkp = link->next; |
261 | if (link->priv) { | 261 | kfree(link->priv); |
262 | kfree(link->priv); | ||
263 | } | ||
264 | kfree(link); | 262 | kfree(link); |
265 | 263 | ||
266 | } /* airo_detach */ | 264 | } /* airo_detach */ |
diff --git a/drivers/net/wireless/atmel.c b/drivers/net/wireless/atmel.c index d57011028b7..1fbe027d26b 100644 --- a/drivers/net/wireless/atmel.c +++ b/drivers/net/wireless/atmel.c | |||
@@ -1653,8 +1653,7 @@ void stop_atmel_card(struct net_device *dev, int freeres) | |||
1653 | unregister_netdev(dev); | 1653 | unregister_netdev(dev); |
1654 | remove_proc_entry("driver/atmel", NULL); | 1654 | remove_proc_entry("driver/atmel", NULL); |
1655 | free_irq(dev->irq, dev); | 1655 | free_irq(dev->irq, dev); |
1656 | if (priv->firmware) | 1656 | kfree(priv->firmware); |
1657 | kfree(priv->firmware); | ||
1658 | if (freeres) { | 1657 | if (freeres) { |
1659 | /* PCMCIA frees this stuff, so only for PCI */ | 1658 | /* PCMCIA frees this stuff, so only for PCI */ |
1660 | release_region(dev->base_addr, 64); | 1659 | release_region(dev->base_addr, 64); |
@@ -2450,8 +2449,7 @@ static int atmel_ioctl(struct net_device *dev, struct ifreq *rq, int cmd) | |||
2450 | break; | 2449 | break; |
2451 | } | 2450 | } |
2452 | 2451 | ||
2453 | if (priv->firmware) | 2452 | kfree(priv->firmware); |
2454 | kfree(priv->firmware); | ||
2455 | 2453 | ||
2456 | priv->firmware = new_firmware; | 2454 | priv->firmware = new_firmware; |
2457 | priv->firmware_length = com.len; | 2455 | priv->firmware_length = com.len; |
diff --git a/drivers/net/wireless/atmel_cs.c b/drivers/net/wireless/atmel_cs.c index ff031a3985b..195cb36619e 100644 --- a/drivers/net/wireless/atmel_cs.c +++ b/drivers/net/wireless/atmel_cs.c | |||
@@ -259,8 +259,7 @@ static void atmel_detach(dev_link_t *link) | |||
259 | 259 | ||
260 | /* Unlink device structure, free pieces */ | 260 | /* Unlink device structure, free pieces */ |
261 | *linkp = link->next; | 261 | *linkp = link->next; |
262 | if (link->priv) | 262 | kfree(link->priv); |
263 | kfree(link->priv); | ||
264 | kfree(link); | 263 | kfree(link); |
265 | } | 264 | } |
266 | 265 | ||
diff --git a/drivers/net/wireless/hostap/hostap_ioctl.c b/drivers/net/wireless/hostap/hostap_ioctl.c index 53f5246c40a..2617d70bcda 100644 --- a/drivers/net/wireless/hostap/hostap_ioctl.c +++ b/drivers/net/wireless/hostap/hostap_ioctl.c | |||
@@ -552,7 +552,6 @@ static int prism2_ioctl_giwaplist(struct net_device *dev, | |||
552 | 552 | ||
553 | kfree(addr); | 553 | kfree(addr); |
554 | kfree(qual); | 554 | kfree(qual); |
555 | |||
556 | return 0; | 555 | return 0; |
557 | } | 556 | } |
558 | 557 | ||
@@ -3081,9 +3080,7 @@ static int prism2_ioctl_priv_download(local_info_t *local, struct iw_point *p) | |||
3081 | ret = local->func->download(local, param); | 3080 | ret = local->func->download(local, param); |
3082 | 3081 | ||
3083 | out: | 3082 | out: |
3084 | if (param != NULL) | 3083 | kfree(param); |
3085 | kfree(param); | ||
3086 | |||
3087 | return ret; | 3084 | return ret; |
3088 | } | 3085 | } |
3089 | #endif /* PRISM2_DOWNLOAD_SUPPORT */ | 3086 | #endif /* PRISM2_DOWNLOAD_SUPPORT */ |
@@ -3890,9 +3887,7 @@ static int prism2_ioctl_priv_hostapd(local_info_t *local, struct iw_point *p) | |||
3890 | } | 3887 | } |
3891 | 3888 | ||
3892 | out: | 3889 | out: |
3893 | if (param != NULL) | 3890 | kfree(param); |
3894 | kfree(param); | ||
3895 | |||
3896 | return ret; | 3891 | return ret; |
3897 | } | 3892 | } |
3898 | 3893 | ||
diff --git a/drivers/net/wireless/prism54/islpci_dev.c b/drivers/net/wireless/prism54/islpci_dev.c index 6c9584a9f28..78bdb359835 100644 --- a/drivers/net/wireless/prism54/islpci_dev.c +++ b/drivers/net/wireless/prism54/islpci_dev.c | |||
@@ -754,8 +754,7 @@ islpci_free_memory(islpci_private *priv) | |||
754 | pci_unmap_single(priv->pdev, buf->pci_addr, | 754 | pci_unmap_single(priv->pdev, buf->pci_addr, |
755 | buf->size, PCI_DMA_FROMDEVICE); | 755 | buf->size, PCI_DMA_FROMDEVICE); |
756 | buf->pci_addr = 0; | 756 | buf->pci_addr = 0; |
757 | if (buf->mem) | 757 | kfree(buf->mem); |
758 | kfree(buf->mem); | ||
759 | buf->size = 0; | 758 | buf->size = 0; |
760 | buf->mem = NULL; | 759 | buf->mem = NULL; |
761 | } | 760 | } |
diff --git a/drivers/net/wireless/prism54/oid_mgt.c b/drivers/net/wireless/prism54/oid_mgt.c index 12123e24b11..eea2f04c8c6 100644 --- a/drivers/net/wireless/prism54/oid_mgt.c +++ b/drivers/net/wireless/prism54/oid_mgt.c | |||
@@ -268,11 +268,10 @@ mgt_clean(islpci_private *priv) | |||
268 | 268 | ||
269 | if (!priv->mib) | 269 | if (!priv->mib) |
270 | return; | 270 | return; |
271 | for (i = 0; i < OID_NUM_LAST; i++) | 271 | for (i = 0; i < OID_NUM_LAST; i++) { |
272 | if (priv->mib[i]) { | 272 | kfree(priv->mib[i]); |
273 | kfree(priv->mib[i]); | 273 | priv->mib[i] = NULL; |
274 | priv->mib[i] = NULL; | 274 | } |
275 | } | ||
276 | kfree(priv->mib); | 275 | kfree(priv->mib); |
277 | priv->mib = NULL; | 276 | priv->mib = NULL; |
278 | } | 277 | } |
diff --git a/drivers/net/wireless/strip.c b/drivers/net/wireless/strip.c index 7bc7fc82312..d25264ba0c0 100644 --- a/drivers/net/wireless/strip.c +++ b/drivers/net/wireless/strip.c | |||
@@ -860,12 +860,9 @@ static int allocate_buffers(struct strip *strip_info, int mtu) | |||
860 | strip_info->mtu = dev->mtu = mtu; | 860 | strip_info->mtu = dev->mtu = mtu; |
861 | return (1); | 861 | return (1); |
862 | } | 862 | } |
863 | if (r) | 863 | kfree(r); |
864 | kfree(r); | 864 | kfree(s); |
865 | if (s) | 865 | kfree(t); |
866 | kfree(s); | ||
867 | if (t) | ||
868 | kfree(t); | ||
869 | return (0); | 866 | return (0); |
870 | } | 867 | } |
871 | 868 | ||
@@ -922,13 +919,9 @@ static int strip_change_mtu(struct net_device *dev, int new_mtu) | |||
922 | printk(KERN_NOTICE "%s: strip MTU changed fom %d to %d.\n", | 919 | printk(KERN_NOTICE "%s: strip MTU changed fom %d to %d.\n", |
923 | strip_info->dev->name, old_mtu, strip_info->mtu); | 920 | strip_info->dev->name, old_mtu, strip_info->mtu); |
924 | 921 | ||
925 | if (orbuff) | 922 | kfree(orbuff); |
926 | kfree(orbuff); | 923 | kfree(osbuff); |
927 | if (osbuff) | 924 | kfree(otbuff); |
928 | kfree(osbuff); | ||
929 | if (otbuff) | ||
930 | kfree(otbuff); | ||
931 | |||
932 | return 0; | 925 | return 0; |
933 | } | 926 | } |
934 | 927 | ||
@@ -2498,18 +2491,13 @@ static int strip_close_low(struct net_device *dev) | |||
2498 | /* | 2491 | /* |
2499 | * Free all STRIP frame buffers. | 2492 | * Free all STRIP frame buffers. |
2500 | */ | 2493 | */ |
2501 | if (strip_info->rx_buff) { | 2494 | kfree(strip_info->rx_buff); |
2502 | kfree(strip_info->rx_buff); | 2495 | strip_info->rx_buff = NULL; |
2503 | strip_info->rx_buff = NULL; | 2496 | kfree(strip_info->sx_buff); |
2504 | } | 2497 | strip_info->sx_buff = NULL; |
2505 | if (strip_info->sx_buff) { | 2498 | kfree(strip_info->tx_buff); |
2506 | kfree(strip_info->sx_buff); | 2499 | strip_info->tx_buff = NULL; |
2507 | strip_info->sx_buff = NULL; | 2500 | |
2508 | } | ||
2509 | if (strip_info->tx_buff) { | ||
2510 | kfree(strip_info->tx_buff); | ||
2511 | strip_info->tx_buff = NULL; | ||
2512 | } | ||
2513 | del_timer(&strip_info->idle_timer); | 2501 | del_timer(&strip_info->idle_timer); |
2514 | return 0; | 2502 | return 0; |
2515 | } | 2503 | } |