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

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Tue, 24 Jul 2007 14:32:44 +0200
From:	Takashi Iwai <tiwai@...e.de>
To:	Adrian Bunk <bunk@...sta.de>
Cc:	perex@...e.cz, linux-kernel@...r.kernel.org
Subject: Re: [2.6 patch] sound/pci/cs46xx/: fix an off-by-one

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@...sta.de>

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@...r.kernel.org
> 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@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/