Re: [PATCH v4 0/5] MR8438: cfgmgr32: Device Query API part 1: Add a basic implementation for DevGetObjects.

[Vibhav Pant (＠vibhavp)]

On Mon Jun 30 13:17:20 2025 +0000, Rémi Bernon wrote:
> Fwiw in general output parameters aren't checked in the error cases,
> because it shouldn't matter what happens to them. For cases where it
> *does* matter (like whether they should be zeroed), the params are set
> before calling to some arbitrary and different value (0xdeadbeef) from
> the expected.
> In particular, at this point these tests succeed already because the
> stub doesn't change the parameter values. Same thing below.
>     hr = DevGetObjects( DevObjectTypeDeviceInterface, DevQueryFlagNone,
> 1, NULL, 0, NULL, &len, &objects );
>     todo_wine ok( hr == HRESULT_FROM_WIN32( ERROR_INVALID_PARAMETER ),
> "got hr %#lx\n", hr );

Gotcha, removed the unnecessary checks in the latest revision, thanks.