setupapi: Implement pSetupGetField, with tests.

James Hawkins truiken at gmail.com
Thu Mar 27 10:44:22 CDT 2008


On Thu, Mar 27, 2008 at 7:06 AM, Paul Vriens <paul.vriens.wine at gmail.com> wrote:
> Hi James,
>
>  I'm looking into a crash of the setupapi/parser test on Vista.
>
>  It crashes in this piece:
>
>   465         ret = HeapFree( GetProcessHeap(), 0, (LPVOID)field );
>   466         ok( !ret, "Expected HeapFree to fail\n" );
>   467         ok( GetLastError() == ERROR_INVALID_PARAMETER,
>   468             "Expected ERROR_INVALID_PARAMETER, got %u\n", GetLastError() );
>
>  when trying the second field.
>
>  It's been added by a patch of yours:
>
>  http://source.winehq.org/git/wine.git/?a=commit;h=7588b669d8331ff64fc3b75cebe40ef518cd0de1
>
>  What's the reason for checking the HeapFree here? Or what's the reason for doing
>  it at all?
>

To show that the value returned is not in the heap.

-- 
James Hawkins



More information about the wine-devel mailing list