Re: setupapi: Implement pSetupGetField, with tests.

[Paul Vriens]

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=7588b669d8331ff64fc3b75ceb...

What's the reason for checking the HeapFree here? Or what's the reason for doing 
it at all?