Re: [PATCH v8 0/2] MR6904: ntdll.dll: Update NtQueryDirectoryFile to align with current Windows behaviour

[Eugene McArdle (＠eugenemcardle)]

On Wed Jan 29 22:46:11 2025 +0000, eric pouech wrote:
> it doesn't make sense to pass &notatestfile when expected status isn't STATUS_SUCCESS
> IOW, all tests with expected_status != STATUS_SUCCESS should have NULL
> as expected filename
> (actually, we could even think that for the expected_status ==
> STATUS_SUCCESS case, the expected filename isn't NULL, but that would
> requires some more work as a NULL mask generates other filenames
> starting with "." and "..")

You're absolutely right, in the case where status isn't STATUS_SUCCESS the name check is never done, so there is no point in passing it in at all. I'll correct this and replace it with NULL.