From 179f3eb8063911c0b731dfd53a0037ccd9d214e2 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Herm=C3=A8s=20B=C3=A9lusca-Ma=C3=AFto?= Date: Tue, 10 Sep 2013 20:45:53 +0000 Subject: [PATCH 1/1] [BASESRV] - Fix header formatting plus windows headers inclusion. - RtlCreateUnicodeString returns booleans, not ntstatuses, so fix the code appropriately. svn path=/trunk/; revision=60012 --- reactos/subsystems/win/basesrv/basesrv.h | 43 ++++++++++++++---------- reactos/subsystems/win/basesrv/init.c | 19 ++++++----- 2 files changed, 35 insertions(+), 27 deletions(-) diff --git a/reactos/subsystems/win/basesrv/basesrv.h b/reactos/subsystems/win/basesrv/basesrv.h index 0a38f2bc506..45a7dcc3ffd 100644 --- a/reactos/subsystems/win/basesrv/basesrv.h +++ b/reactos/subsystems/win/basesrv/basesrv.h @@ -12,11 +12,19 @@ #pragma once /* PSDK/NDK Headers */ +#include #define WIN32_NO_STATUS -#include +#define _INC_WINDOWS +#define COM_NO_WINDOWS_H +#include +#include +#include #define NTOS_MODE_USER #include +/* PSEH for SEH Support */ +#include + /* CSRSS Header */ #include @@ -31,25 +39,24 @@ typedef struct _BASESRV_KERNEL_IMPORTS } BASESRV_KERNEL_IMPORTS, *PBASESRV_KERNEL_IMPORTS; /* FIXME: BASENLS.H */ -typedef NTSTATUS(*WINAPI POPEN_DATA_FILE) (HANDLE hFile, - PWCHAR FileName - ); - -typedef BOOL(*WINAPI PGET_CP_FILE_NAME_FROM_REGISTRY)(UINT CodePage, - LPWSTR FileName, - ULONG FileNameSize - ); - -typedef BOOL(*WINAPI PGET_NLS_SECTION_NAME)(UINT CodePage, - UINT Base, - ULONG Unknown, - LPWSTR BaseName, - LPWSTR Result, - ULONG ResultSize - ); +typedef NTSTATUS(*WINAPI POPEN_DATA_FILE)(HANDLE hFile, + PWCHAR FileName); + +typedef BOOL(*WINAPI PGET_CP_FILE_NAME_FROM_REGISTRY)(UINT CodePage, + LPWSTR FileName, + ULONG FileNameSize); + +typedef BOOL(*WINAPI PGET_NLS_SECTION_NAME)(UINT CodePage, + UINT Base, + ULONG Unknown, + LPWSTR BaseName, + LPWSTR Result, + ULONG ResultSize); typedef BOOL(*WINAPI PVALIDATE_LOCALE)(IN ULONG LocaleId); -typedef NTSTATUS(*WINAPI PCREATE_NLS_SECURTY_DESCRIPTOR)(IN PVOID Buffer, IN ULONG BufferSize, IN ULONG AceType); +typedef NTSTATUS(*WINAPI PCREATE_NLS_SECURTY_DESCRIPTOR)(IN PVOID Buffer, + IN ULONG BufferSize, + IN ULONG AceType); /* Globals */ extern HANDLE BaseSrvHeap; diff --git a/reactos/subsystems/win/basesrv/init.c b/reactos/subsystems/win/basesrv/init.c index 28680f2fb7e..6489ca8c276 100644 --- a/reactos/subsystems/win/basesrv/init.c +++ b/reactos/subsystems/win/basesrv/init.c @@ -280,6 +280,7 @@ NTAPI BaseInitializeStaticServerData(IN PCSR_SERVER_DLL LoadedServerDll) { NTSTATUS Status; + BOOLEAN Success; WCHAR Buffer[MAX_PATH]; PWCHAR HeapBuffer; UNICODE_STRING SystemRootString; @@ -327,21 +328,21 @@ BaseInitializeStaticServerData(IN PCSR_SERVER_DLL LoadedServerDll) /* Create the base directory */ Buffer[SystemRootString.Length / sizeof(WCHAR)] = UNICODE_NULL; - Status = RtlCreateUnicodeString(&BaseSrvWindowsDirectory, - SystemRootString.Buffer); - ASSERT(NT_SUCCESS(Status)); + Success = RtlCreateUnicodeString(&BaseSrvWindowsDirectory, + SystemRootString.Buffer); + ASSERT(Success); /* Create the system directory */ wcscat(SystemRootString.Buffer, L"\\System32"); - Status = RtlCreateUnicodeString(&BaseSrvWindowsSystemDirectory, - SystemRootString.Buffer); - ASSERT(NT_SUCCESS(Status)); + Success = RtlCreateUnicodeString(&BaseSrvWindowsSystemDirectory, + SystemRootString.Buffer); + ASSERT(Success); /* Create the kernel32 path */ wcscat(SystemRootString.Buffer, L"\\kernel32.dll"); - Status = RtlCreateUnicodeString(&BaseSrvKernel32DllPath, - SystemRootString.Buffer); - ASSERT(NT_SUCCESS(Status)); + Success = RtlCreateUnicodeString(&BaseSrvKernel32DllPath, + SystemRootString.Buffer); + ASSERT(Success); /* FIXME: Check Session ID */ wcscpy(Buffer, L"\\BaseNamedObjects"); -- 2.17.1