mirror of https://gitee.com/openkylin/linux.git
[ALSA] NULL pointer dereference in sound/synth/emux/soundfont.c
this is about coverity id #100. It seems the if statement is negated, since the else branch calls remove_info() with sflist->currsf as a parameter where it gets dereferenced. Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de> Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
parent
f26eb78fcf
commit
d20cad602f
|
@ -195,7 +195,7 @@ snd_soundfont_load(struct snd_sf_list *sflist, const void __user *data,
|
|||
break;
|
||||
case SNDRV_SFNT_REMOVE_INFO:
|
||||
/* patch must be opened */
|
||||
if (sflist->currsf) {
|
||||
if (!sflist->currsf) {
|
||||
snd_printk("soundfont: remove_info: patch not opened\n");
|
||||
rc = -EINVAL;
|
||||
} else {
|
||||
|
|
Loading…
Reference in New Issue