diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2015-01-03 22:55:54 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2015-01-04 15:13:45 +0100 |
commit | ff6defa6a8fae12205d64f55db395b1fcf35af8e (patch) | |
tree | a08b5c63a0c7e21f955e254178dc53ea0d7b6e3f /sound/pci/ad1889.c | |
parent | b76c0e95ab0f535e210df588b8ac42dd75b31c75 (diff) | |
download | op-kernel-dev-ff6defa6a8fae12205d64f55db395b1fcf35af8e.zip op-kernel-dev-ff6defa6a8fae12205d64f55db395b1fcf35af8e.tar.gz |
ALSA: Deletion of checks before the function call "iounmap"
The iounmap() function performs also input parameter validation.
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: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci/ad1889.c')
-rw-r--r-- | sound/pci/ad1889.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/sound/pci/ad1889.c b/sound/pci/ad1889.c index 547ee30..0de3129 100644 --- a/sound/pci/ad1889.c +++ b/sound/pci/ad1889.c @@ -853,12 +853,9 @@ snd_ad1889_free(struct snd_ad1889 *chip) free_irq(chip->irq, chip); skip_hw: - if (chip->iobase) - iounmap(chip->iobase); - + iounmap(chip->iobase); pci_release_regions(chip->pci); pci_disable_device(chip->pci); - kfree(chip); return 0; } |