Re: kernel32: Remove unused variable

[Vitaliy Margolen]

On 09/19/2010 01:47 AM, Luca Bennati wrote:
> Found with scan-build provided by clang/llvm
>
> diff --git a/dlls/kernel32/virtual.c b/dlls/kernel32/virtual.c
> index dcd5b85..a10bb6c 100644
> --- a/dlls/kernel32/virtual.c
> +++ b/dlls/kernel32/virtual.c
> @@ -648,17 +648,13 @@ BOOL WINAPI IsBadReadPtr( LPCVOID ptr, UINT size )
>      __TRY
>      {
>          volatile const char *p = ptr;
> -
>    char dummy;
>    UINT count = size;
>
>    while (count > page_size)
>    {
> -
>        dummy = *p;
>        p += page_size;
>        count -= page_size;
>    }
> -
>    dummy = p[0];
> -
>    dummy = p[count - 1];

You can't do that. That's the whole point of this function - to test if 
particular address is readable.

Vitaliy.