[PATCH 1/4] dlls/ntdll: Add handle count to NtQuerySystemInformation

Vitaliy Margolen wine-devel at kievinfo.com
Mon Jun 20 08:52:06 CDT 2011


On 06/20/2011 05:28 AM, Austin Lund wrote:
> Thanks for the review.  Does this patch address these concerns correctly?
No quite there yet:

> +            SERVER_START_REQ( create_snapshot )
You not checking if this call succeeded or not.

> +                while (sizeof(ULONG) + sizeof(SYSTEM_HANDLE_ENTRY)*shi->Count >= len)
You don't need a loop to calculate new size.

> +                    shi = RtlReAllocateHeap(GetProcessHeap(), HEAP_ZERO_MEMORY, shi, len);
> +                    if (shi == NULL)
> +                        return STATUS_NO_MEMORY;
You leaking old shi here. Also why do you need to zero allocated memory if 
you assigning all of it valid values?

Vitaliy.



More information about the wine-devel mailing list