Paul Vriens wrote:
Hi,
Hi Paul,
the first bigger change to SYSTEM_PROCESS_INFORMATION. The structure now contains VM_COUNTERS (which had to move up in the file) instead of all the single (wrong) items, and IO_COUNTERS. I also added 2 extra FILETIME items (well actually I changed dwUnknown2[5]).
This structure now has the correct size for W2K/WinXP and W2K3. NT doesn't have the IO_COUNTERS part, but for now I'm not to worried about dealing with that (unless somebody tells me to).
I also had to change ntdll/nt.c because of this. I removed all the single sets to 0 as this is done by the memset anyway.
Next patches will change the struct (and related files) to use UNICODE_STRING for the process name, but I'm having a few issues with that.
Next patches will also contain tests to prove the above. Can anybody (who's not tired of reading until now) tell me if we really need the __WINESRC__ in that structure, the Windows structure shown is wrong anyway
We shouldn't need the __WINESRC__ ifdef. No code that is designed to be portable to Microsoft's headers should be using winternl.h.
and how can I check 'my' struct in a test (for now I've copied it to the test and renamed it to something non-existent)?
I think that is the only way to do it.
-
Robert Shearman