diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-02-04 12:32:14 +0100 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2015-02-05 09:25:10 +0100 |
commit | 2d4ad4f6903f3a7b4eccb6e7eeb82f22edd648ea (patch) | |
tree | cdb8e6c10ebe0d53b6dcb46835498129be842680 | |
parent | cxgb4: Delete an unnecessary check before the function call "release_firmware" (diff) | |
download | linux-2d4ad4f6903f3a7b4eccb6e7eeb82f22edd648ea.tar.xz linux-2d4ad4f6903f3a7b4eccb6e7eeb82f22edd648ea.zip |
myri10ge: Delete an unnecessary check before the function call "kfree"
The kfree() function tests whether its argument is NULL and then
returns immediately. Thus the test around the call is not needed.
This issue was detected by using the Coccinelle software.
Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/ethernet/myricom/myri10ge/myri10ge.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ethernet/myricom/myri10ge/myri10ge.c b/drivers/net/ethernet/myricom/myri10ge/myri10ge.c index 71af98bb72cb..1412f5af05ec 100644 --- a/drivers/net/ethernet/myricom/myri10ge/myri10ge.c +++ b/drivers/net/ethernet/myricom/myri10ge/myri10ge.c @@ -4226,8 +4226,7 @@ static void myri10ge_remove(struct pci_dev *pdev) mtrr_del(mgp->mtrr, mgp->iomem_base, mgp->board_span); #endif myri10ge_free_slices(mgp); - if (mgp->msix_vectors != NULL) - kfree(mgp->msix_vectors); + kfree(mgp->msix_vectors); dma_free_coherent(&pdev->dev, sizeof(*mgp->cmd), mgp->cmd, mgp->cmd_bus); |