[NTOS]
authorHermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>
Fri, 4 Nov 2016 17:52:32 +0000 (17:52 +0000)
committerHermès Bélusca-Maïto <hermes.belusca-maito@reactos.org>
Fri, 4 Nov 2016 17:52:32 +0000 (17:52 +0000)
commit15d77be6fd467bfed4a4e4f98a7432b935486efb
tree357ca560c8e0749dd1a17235039f9b52edd25c49
parent3430268b90304cc0cf0f8413a49b137848614ab0
[NTOS]
- accesschk.c: Remove redundant SepAccessCheck/SepAccessCheckEx pair of private functions; instead just rename SepAccessCheckEx into SepAccessCheck and use it directly in the code. NOTE: SepAccessCheck is *incomplete* (in particular it doesn't retrieve the information needed to initialize the 'Privileges' parameter).
- sid.c: Comments formatting fix.
- token.c:
  * Finish to implement SeQueryInformationToken . This function is really the same as NtQueryInformationToken but without all the stuff needed for user-mode buffer access protection.
  * Some code simplifications in NtQueryInformationToken.
  I need this to fix a "FIXME: Use SeQueryInformationToken" in some code I'm also fixing (& commit later).

[NDK]: Fix parameter types and add annotations to RtlCopySidAndAttributesArray.

[KMTESTS:NTOS_SE]
- Reenable the 'SeQueryInfoToken' test.
- Show that SeQueryInformationToken doesn't support 4 token information classes, which are supported only by NtQueryInformationToken.
- Fix calling of SeAccessCheck. In particular the 'Privileges' parameter is not allocated by the caller, but instead is allocated by SeAccessCheck *and* returned to the caller (who then must free the buffer using SeFreePrivileges). This fixes the encountered BSODs that leaded to disabling preventively the test in r59178.
- Minor code cleaning.

svn path=/trunk/; revision=73122
reactos/ntoskrnl/se/accesschk.c
reactos/ntoskrnl/se/sid.c
reactos/ntoskrnl/se/token.c
reactos/sdk/include/ndk/rtlfuncs.h
reactos/sdk/lib/rtl/sid.c
rostests/kmtests/kmtest_drv/testlist.c
rostests/kmtests/ntos_se/SeQueryInfoToken.c