diff options
| author | Markus Elfring <elfring@users.sourceforge.net> | 2015-01-03 15:10:52 +0100 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2015-01-04 15:11:56 +0100 |
| commit | d1f45e6037f1195d10cf363e94a08ee2f10f9b41 (patch) | |
| tree | 4e68e24f1ef8deb949c21bf2caa2be1f8531bbff | |
| parent | 57dca36ee25bcab20a71705175c00b6f114cdab4 (diff) | |
| download | op-kernel-dev-d1f45e6037f1195d10cf363e94a08ee2f10f9b41.zip op-kernel-dev-d1f45e6037f1195d10cf363e94a08ee2f10f9b41.tar.gz | |
sound: oss: Deletion of unnecessary checks before the function call "vfree"
The vfree() 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>
| -rw-r--r-- | sound/oss/pss.c | 2 | ||||
| -rw-r--r-- | sound/oss/trix.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/sound/oss/pss.c b/sound/oss/pss.c index ca0d6e9..81314f9 100644 --- a/sound/oss/pss.c +++ b/sound/oss/pss.c @@ -1228,7 +1228,7 @@ static void __exit cleanup_pss(void) { if(!pss_no_sound) { - if(fw_load && pss_synth) + if (fw_load) vfree(pss_synth); if(pssmss) unload_pss_mss(&cfg2); diff --git a/sound/oss/trix.c b/sound/oss/trix.c index 944e0c0..3c494dc 100644 --- a/sound/oss/trix.c +++ b/sound/oss/trix.c @@ -487,7 +487,7 @@ static int __init init_trix(void) static void __exit cleanup_trix(void) { - if (fw_load && trix_boot) + if (fw_load) vfree(trix_boot); if (sb) unload_trix_sb(&cfg2); |
