Nikolay Sivov (@nsivov) commented about dlls/kernel32/tests/actctx.c:
static BOOL (WINAPI *pQueryActCtxSettingsW)(DWORD,HANDLE,LPCWSTR,LPCWSTR,LPWSTR,SIZE_T,SIZE_T*);
+static NTSTATUS (NTAPI *pRtlActivateActivationContext)(ULONG,HANDLE,ULONG_PTR *); +static NTSTATUS (NTAPI *pRtlActivateActivationContextEx)(ULONG,TEB *,HANDLE,ULONG_PTR *); static NTSTATUS(NTAPI *pRtlFindActivationContextSectionString)(DWORD,const GUID *,ULONG,PUNICODE_STRING,PACTCTX_SECTION_KEYED_DATA);
We should stop using HANDLE here, it's causing confusion, and mistakes like wrong release call in the previous version. We'll keep a HANDLE type in kernelbase of course as it's public, but for Rtl* functions it's fine to use structure pointers. This could be done as a separate commit or MR.