diff mbox series

[-next] ALSA: nm256: Fix error return code in snd_nm256_create()

Message ID 20210720135237.3424521-1-yangyingliang@huawei.com (mailing list archive)
State New, archived
Headers show
Series [-next] ALSA: nm256: Fix error return code in snd_nm256_create() | expand

Commit Message

Yang Yingliang July 20, 2021, 1:52 p.m. UTC
If pci_request_regions() fails, it should return error
code in snd_nm256_create().

Reported-by: Hulk Robot <hulkci@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
 sound/pci/nm256/nm256.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Takashi Iwai July 20, 2021, 2:08 p.m. UTC | #1
On Tue, 20 Jul 2021 15:52:37 +0200,
Yang Yingliang wrote:
> 
> If pci_request_regions() fails, it should return error
> code in snd_nm256_create().
> 
> Reported-by: Hulk Robot <hulkci@huawei.com>
> Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>

Thanks, applied.


Takashi
diff mbox series

Patch

diff --git a/sound/pci/nm256/nm256.c b/sound/pci/nm256/nm256.c
index a54b9b26a0c2..c9c178504959 100644
--- a/sound/pci/nm256/nm256.c
+++ b/sound/pci/nm256/nm256.c
@@ -1478,7 +1478,8 @@  snd_nm256_create(struct snd_card *card, struct pci_dev *pci)
 	chip->buffer_addr = pci_resource_start(pci, 0);
 	chip->cport_addr = pci_resource_start(pci, 1);
 
-	if (pci_request_regions(pci, card->driver))
+	err = pci_request_regions(pci, card->driver);
+	if (err < 0)
 		return err;
 
 	/* Init the memory port info.  */