From: Adrian Bunk This patch fixes off-by-one errors found by the Coverity checker. Signed-off-by: Adrian Bunk Signed-off-by: Andrew Morton --- sound/pci/rme9652/hdspm.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff -puN sound/pci/rme9652/hdspm.c~sound-pci-rme9652-hdspmc-fix-off-by-one-errors sound/pci/rme9652/hdspm.c --- devel/sound/pci/rme9652/hdspm.c~sound-pci-rme9652-hdspmc-fix-off-by-one-errors 2006-03-11 01:09:49.000000000 -0800 +++ devel-akpm/sound/pci/rme9652/hdspm.c 2006-03-11 01:09:49.000000000 -0800 @@ -474,7 +474,7 @@ static inline unsigned int hdspm_read(st static inline int hdspm_read_in_gain(struct hdspm * hdspm, unsigned int chan, unsigned int in) { - if (chan > HDSPM_MIXER_CHANNELS || in > HDSPM_MIXER_CHANNELS) + if (chan >= HDSPM_MIXER_CHANNELS || in >= HDSPM_MIXER_CHANNELS) return 0; return hdspm->mixer->ch[chan].in[in]; @@ -483,7 +483,7 @@ static inline int hdspm_read_in_gain(str static inline int hdspm_read_pb_gain(struct hdspm * hdspm, unsigned int chan, unsigned int pb) { - if (chan > HDSPM_MIXER_CHANNELS || pb > HDSPM_MIXER_CHANNELS) + if (chan >= HDSPM_MIXER_CHANNELS || pb >= HDSPM_MIXER_CHANNELS) return 0; return hdspm->mixer->ch[chan].pb[pb]; } _