linux/sound/pci/oxygen
Jesper Juhl 233d84c46c ALSA: Xonar, CS43xx: Don't overrun static array
'cs4398_regs' in 'struct xonar_cs43xx' is an array of 'u8' with a size of
8. So, this code in sound/pci/oxygen/xonar_cs43xx.c::dump_d1_registers()

    		for (i = 2; i <= 8; ++i)
	  			snd_iprintf(buffer, " %02x", data->cs4398_regs[i]);

will overrun the array when 'i == 8'.

I guess that what's needed to fix it is the trivial patch below, but I
must admit that I have no idea about this code, so I may very well be
wrong. Additionally, I have no way to actually test this, so all I know is
that the below compiles. Someone who actually knows this code should take
a look before anything is comitted - consider the below (not much more
than) a bug report.

Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Acked-by: Clemens Ladisch <clemens@ladisch.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
2011-01-22 17:24:30 +01:00
..
Makefile ALSA: oxygen: add Xonar DG support 2011-01-10 16:46:46 +01:00
ak4396.h [ALSA] add TempoTec HiFier driver 2008-01-31 17:30:15 +01:00
cm9780.h [ALSA] oxygen: add more symbols 2008-01-31 17:30:07 +01:00
cs2000.h sound: virtuoso: fix Xonar Essence ST support 2009-09-28 11:53:51 +02:00
cs4245.h ALSA: oxygen: add Xonar DG support 2011-01-10 16:46:46 +01:00
cs4362a.h [ALSA] virtuoso: add Xonar DX support 2008-04-24 12:00:32 +02:00
cs4398.h [ALSA] virtuoso: add Xonar DX support 2008-04-24 12:00:32 +02:00
oxygen.c ALSA: oxygen: add S/PDIF source selection for Claro cards 2011-01-11 17:54:15 +01:00
oxygen.h ALSA: oxygen: do not show chip revision in card longname 2011-01-10 16:47:24 +01:00
oxygen_io.c ALSA: oxygen: fix SPI clocks slower than 6.25 MHz 2011-01-10 16:46:00 +01:00
oxygen_lib.c ALSA: oxygen: do not show chip revision in card longname 2011-01-10 16:47:24 +01:00
oxygen_mixer.c ALSA: oxygen: add digital input validity check switch 2011-01-10 16:47:17 +01:00
oxygen_pcm.c ALSA: oxygen: simplify model-specific MCLK handling 2011-01-10 16:46:31 +01:00
oxygen_regs.h ALSA: oxygen: simplify model-specific MCLK handling 2011-01-10 16:46:31 +01:00
pcm1796.h [ALSA] virtuoso: move PCM1796 symbols to a header file 2008-04-24 12:00:27 +02:00
virtuoso.c ALSA: virtuoso: add HDAV1.3 Slim PCI ID 2010-10-05 18:11:19 +02:00
wm8766.h sound: virtuoso: add Xonar DS support 2010-01-18 16:38:41 +01:00
wm8776.h sound: virtuoso: add Xonar DS support 2010-01-18 16:38:41 +01:00
wm8785.h [ALSA] oxygen: move WM8785 symbols to a header file 2008-04-24 12:00:27 +02:00
xonar.h ALSA: virtuoso: add HDMI enable switch for HDAV1.3 2010-12-06 14:47:58 +01:00
xonar_cs43xx.c ALSA: Xonar, CS43xx: Don't overrun static array 2011-01-22 17:24:30 +01:00
xonar_dg.c ALSA: oxygen: Xonar DG: fix CS4245 register writes 2011-01-14 09:50:01 +01:00
xonar_dg.h ALSA: oxygen: add Xonar DG support 2011-01-10 16:46:46 +01:00
xonar_hdmi.c ALSA: virtuoso: add Xonar HDAV1.3 Slim support 2011-01-10 16:46:49 +01:00
xonar_lib.c ALSA: virtuoso: add HDMI enable switch for HDAV1.3 2010-12-06 14:47:58 +01:00
xonar_pcm179x.c ALSA: core, oxygen, virtuoso: add an enum control info helper 2011-01-10 16:46:53 +01:00
xonar_wm87x6.c ALSA: core, oxygen, virtuoso: add an enum control info helper 2011-01-10 16:46:53 +01:00