diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2014-11-20 15:50:43 +0300 |
|---|---|---|
| committer | Brian Norris <computersforpeace@gmail.com> | 2014-11-26 09:44:10 +0300 |
| commit | 05a221bb1f49e6eebc9a3858cb45506f403b3ab6 (patch) | |
| tree | aa85b86eb0cc5d7aca9e6da39394157d8491f87c /drivers/mtd/chips/cfi_cmdset_0001.c | |
| parent | 775a9134f4398ca98a10af8cc3cf9b664017267f (diff) | |
| download | linux-05a221bb1f49e6eebc9a3858cb45506f403b3ab6.tar.xz | |
mtd: delete unnecessary checks before two function calls
The functions kfree() and pci_dev_put() test whether their argument is NULL
and then return 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: Brian Norris <computersforpeace@gmail.com>
Diffstat (limited to 'drivers/mtd/chips/cfi_cmdset_0001.c')
| -rw-r--r-- | drivers/mtd/chips/cfi_cmdset_0001.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/mtd/chips/cfi_cmdset_0001.c b/drivers/mtd/chips/cfi_cmdset_0001.c index 3096f3ded3ad..286b97a304cf 100644 --- a/drivers/mtd/chips/cfi_cmdset_0001.c +++ b/drivers/mtd/chips/cfi_cmdset_0001.c @@ -2654,8 +2654,7 @@ static void cfi_intelext_destroy(struct mtd_info *mtd) kfree(cfi); for (i = 0; i < mtd->numeraseregions; i++) { region = &mtd->eraseregions[i]; - if (region->lockmap) - kfree(region->lockmap); + kfree(region->lockmap); } kfree(mtd->eraseregions); } |