diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2014-11-22 19:29:01 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2014-11-22 21:48:33 +0100 |
commit | 4fc390a198c41a007c02f82f41ad100c7c67b55e (patch) | |
tree | ed6ed7da77182dd1d1509eef9e6ac9b32fa386af /sound | |
parent | ALSA: sb: Deletion of unnecessary checks before two function calls (diff) | |
download | linux-4fc390a198c41a007c02f82f41ad100c7c67b55e.tar.xz linux-4fc390a198c41a007c02f82f41ad100c7c67b55e.zip |
sound: oss: uart401: remove unneeded NULL check
"devc" can't be NULL here so there is no need to check. Also I removed
the "devc = NULL" assignment because devc is stored on stack so it's
a no-op.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
-rw-r--r-- | sound/oss/uart401.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/sound/oss/uart401.c b/sound/oss/uart401.c index 279bc565ac7e..dae4d4344407 100644 --- a/sound/oss/uart401.c +++ b/sound/oss/uart401.c @@ -412,13 +412,10 @@ void unload_uart401(struct address_info *hw_config) if (!devc->share_irq) free_irq(devc->irq, devc); - if (devc) - { - kfree(midi_devs[devc->my_dev]->converter); - kfree(midi_devs[devc->my_dev]); - kfree(devc); - devc = NULL; - } + kfree(midi_devs[devc->my_dev]->converter); + kfree(midi_devs[devc->my_dev]); + kfree(devc); + /* This kills midi_devs[x] */ sound_unload_mididev(hw_config->slots[4]); } |