Re: [2.6 patch] sound/pci/cs46xx/: fix an off-by-one

[Takashi Iwai]

At Mon, 23 Jul 2007 16:51:16 +0200,
Adrian Bunk wrote:
> 
> This patch fixes an off-by-one in a snd_assert() spotted by the 
> Coverity checker.
> 
> Signed-off-by: Adrian Bunk <bunk@xxxxxxxxx>

Applied it to ALSA tree.  Thanks.


Takashi

> 
> ---
> --- linux-2.6.22-rc6-mm1/sound/pci/cs46xx/dsp_spos_scb_lib.c.old	2007-07-23 15:33:17.000000000 +0200
> +++ linux-2.6.22-rc6-mm1/sound/pci/cs46xx/dsp_spos_scb_lib.c	2007-07-23 15:33:37.000000000 +0200
> @@ -1480,7 +1480,7 @@ void cs46xx_dsp_destroy_pcm_channel (str
>  	if (!pcm_channel->src_scb->ref_count) {
>  		cs46xx_dsp_remove_scb(chip,pcm_channel->src_scb);
>  
> -		snd_assert (pcm_channel->src_slot >= 0 && pcm_channel->src_slot <= DSP_MAX_SRC_NR,
> +		snd_assert (pcm_channel->src_slot >= 0 && pcm_channel->src_slot < DSP_MAX_SRC_NR,
>  			    return );
>  
>  		ins->src_scb_slots[pcm_channel->src_slot] = 0;
> 
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
> 
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/