Re: [PATCH] ALSA: off by 1 in snd_emu10k1_wait()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At Thu, 16 Apr 2009 23:54:04 +0200,
Roel Kluin wrote:
> 
> Is this useful? It was in Linus' git tree.

Well, I guess it won't change any real behavior, but a bug is a bug :)
Applied now.

Thanks,

Takashi



> ----------------------------->8-----------------8<------------------------------
> With `while (count++ < 16384)' count reaches 16385.
> 
> Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx>
> ---
> diff --git a/sound/pci/emu10k1/io.c b/sound/pci/emu10k1/io.c
> index 4bfc31d..c1a5aa1 100644
> --- a/sound/pci/emu10k1/io.c
> +++ b/sound/pci/emu10k1/io.c
> @@ -490,7 +490,7 @@ void snd_emu10k1_wait(struct snd_emu10k1 *emu, unsigned int wait)
>  			if (newtime != curtime)
>  				break;
>  		}
> -		if (count >= 16384)
> +		if (count > 16384)
>  			break;
>  		curtime = newtime;
>  	}
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel@xxxxxxxxxxxxxxxx
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux