Commit 2462321b authored Apr 10, 2006 by Eric Sesterhenn Committed by Jaroslav Kysela Apr 12, 2006

[ALSA] Overrun in sound/pci/au88x0/au88x0_pcm.c



Modules: au88x0 driver

since idx is used as an index for vortex_pcm_prettyname[VORTEX_PCM_LAST],
it should not be equal to VORTEX_PCM_LAST. This fixes coverity bug id #572

Signed-off-by: Eric Sesterhenn <snakebyte@gmx.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>

parent 99e80e4d

sound/pci/au88x0/au88x0_pcm.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -506,7 +506,7 @@ static int __devinit snd_vortex_new_pcm(vortex_t * chip, int idx, int nr)
		int i;
		int err, nr_capt;

		if ((chip == 0) \|\| (idx < 0) \|\| (idx > VORTEX_PCM_LAST))
		if ((chip == 0) \|\| (idx < 0) \|\| (idx >= VORTEX_PCM_LAST))
		return -ENODEV;

		/* idx indicates which kind of PCM device. ADB, SPDIF, I2S and A3D share the