diff options
author | Xiaoke Wang <xkernel.wang@foxmail.com> | 2022-03-04 09:38:20 +0100 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2022-03-04 09:51:37 +0100 |
commit | d7f15befac809ba365742464e1b0bebf07149c58 (patch) | |
tree | 05f63aa0ba5dcf5795cf38c02593d7fca6016ae4 /sound/pci/lola | |
parent | ALSA: echoaudio: remove redundant assignment to variable bytes (diff) | |
download | linux-d7f15befac809ba365742464e1b0bebf07149c58.tar.xz linux-d7f15befac809ba365742464e1b0bebf07149c58.zip |
ALSA: lola: add a check for the return of vmalloc()
vmalloc() is a memory allocation function which can return NULL when
some internal memory errors happen. So it is better to check the return
of it to catch the error in time.
Signed-off-by: Xiaoke Wang <xkernel.wang@foxmail.com>
Link: https://lore.kernel.org/r/tencent_4221FC4089F6DF01C48F192E5784038BA205@qq.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci/lola')
-rw-r--r-- | sound/pci/lola/lola_mixer.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/sound/pci/lola/lola_mixer.c b/sound/pci/lola/lola_mixer.c index e2c8f1417001..6b162489cb5f 100644 --- a/sound/pci/lola/lola_mixer.c +++ b/sound/pci/lola/lola_mixer.c @@ -121,6 +121,8 @@ int lola_init_mixer_widget(struct lola *chip, int nid) /* reserve memory to copy mixer data for sleep mode transitions */ chip->mixer.array_saved = vmalloc(sizeof(struct lola_mixer_array)); + if (!chip->mixer.array_saved) + return -ENOMEM; /* mixer matrix sources are physical input data and play streams */ chip->mixer.src_stream_outs = chip->pcm[PLAY].num_streams; |