Re: [PATCH - asihpi 4/5] add extra checks to control cache

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

 



At Fri, 18 Jul 2008 18:00:18 +1200,
linux@xxxxxxxxxxxxxxxx wrote:
> 
> @@ -486,14 +487,16 @@ short HpiCheckControlCache(
>  					&aDesc[dwIndex].pData[dwOffset], 8);
>  				nRemainingChars =
>  					nStringLength - dwOffset - 8;
> -				if (nRemainingChars < 0)
> +				if (nRemainingChars < 0) {
>  					nRemainingChars = 0;
> +					phr->u.cu.chars8.szData[8 +
> +						nRemainingChars] = 0;
> +				}

This change looks incorrect.  szData is 8-bytes array, and you reset
nRemainigChars to 0.  Also, not sure what this zero-clear actually
does...


Takashi
_______________________________________________
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