From 33bacef5b3af5c2a7352889025520b0125943687 Mon Sep 17 00:00:00 2001 From: Timo Kreuzer Date: Sat, 15 Dec 2012 13:58:24 +0000 Subject: [PATCH] [RTL] Formatting, no code change svn path=/trunk/; revision=57921 --- reactos/lib/rtl/handle.c | 260 +++++++++++++++++++++------------------ 1 file changed, 137 insertions(+), 123 deletions(-) diff --git a/reactos/lib/rtl/handle.c b/reactos/lib/rtl/handle.c index f9170863668..4209a0033b7 100644 --- a/reactos/lib/rtl/handle.c +++ b/reactos/lib/rtl/handle.c @@ -2,7 +2,7 @@ * PROJECT: ReactOS system libraries * PURPOSE: Handle table * FILE: lib/rtl/handle.c - * PROGRAMER: Eric Kohl + * PROGRAMMER: Eric Kohl */ /* INCLUDES *****************************************************************/ @@ -14,182 +14,196 @@ /* GLOBALS ******************************************************************/ -VOID NTAPI -RtlInitializeHandleTable(ULONG TableSize, - ULONG HandleSize, - PRTL_HANDLE_TABLE HandleTable) +VOID +NTAPI +RtlInitializeHandleTable( + ULONG TableSize, + ULONG HandleSize, + PRTL_HANDLE_TABLE HandleTable) { - /* initialize handle table */ - memset(HandleTable, - 0, - sizeof(RTL_HANDLE_TABLE)); - HandleTable->MaximumNumberOfHandles = TableSize; - HandleTable->SizeOfHandleTableEntry = HandleSize; + /* Initialize handle table */ + memset(HandleTable, 0, sizeof(RTL_HANDLE_TABLE)); + HandleTable->MaximumNumberOfHandles = TableSize; + HandleTable->SizeOfHandleTableEntry = HandleSize; } /* * @implemented */ -VOID NTAPI -RtlDestroyHandleTable(PRTL_HANDLE_TABLE HandleTable) +VOID +NTAPI +RtlDestroyHandleTable( + PRTL_HANDLE_TABLE HandleTable) { - PVOID ArrayPointer; - SIZE_T ArraySize = 0; + PVOID ArrayPointer; + SIZE_T ArraySize = 0; - /* free handle array */ - if (HandleTable->CommittedHandles) - { + /* free handle array */ + if (HandleTable->CommittedHandles) + { ArrayPointer = (PVOID)HandleTable->CommittedHandles; NtFreeVirtualMemory(NtCurrentProcess(), - &ArrayPointer, - &ArraySize, - MEM_RELEASE); - } + &ArrayPointer, + &ArraySize, + MEM_RELEASE); + } } /* * @implemented */ -PRTL_HANDLE_TABLE_ENTRY NTAPI -RtlAllocateHandle(PRTL_HANDLE_TABLE HandleTable, - PULONG Index) +PRTL_HANDLE_TABLE_ENTRY +NTAPI +RtlAllocateHandle( + PRTL_HANDLE_TABLE HandleTable, + PULONG Index) { - PRTL_HANDLE_TABLE_ENTRY *pp_new, *pph, ph; - NTSTATUS Status; - PRTL_HANDLE_TABLE_ENTRY retval; - PVOID ArrayPointer; - SIZE_T ArraySize; - - pp_new = &HandleTable->FreeHandles; - - if (HandleTable->FreeHandles == NULL) - { - /* no free handle available */ - if (HandleTable->UnCommittedHandles == NULL) - { - /* allocate handle array */ - ArraySize = HandleTable->SizeOfHandleTableEntry * HandleTable->MaximumNumberOfHandles; - ArrayPointer = NULL; - - /* FIXME - only reserve handles here! */ - Status = NtAllocateVirtualMemory(NtCurrentProcess(), - (PVOID*)&ArrayPointer, - 0, - &ArraySize, - MEM_RESERVE | MEM_COMMIT, - PAGE_READWRITE); - if (!NT_SUCCESS(Status)) - return NULL; - - /* update handle array pointers */ - HandleTable->FreeHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; - HandleTable->MaxReservedHandles = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)ArrayPointer + ArraySize); - HandleTable->CommittedHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; - HandleTable->UnCommittedHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; - } + PRTL_HANDLE_TABLE_ENTRY *pp_new, *pph, ph; + NTSTATUS Status; + PRTL_HANDLE_TABLE_ENTRY retval; + PVOID ArrayPointer; + SIZE_T ArraySize; + + pp_new = &HandleTable->FreeHandles; + + if (HandleTable->FreeHandles == NULL) + { + /* no free handle available */ + if (HandleTable->UnCommittedHandles == NULL) + { + /* allocate handle array */ + ArraySize = HandleTable->SizeOfHandleTableEntry * HandleTable->MaximumNumberOfHandles; + ArrayPointer = NULL; + + /* FIXME - only reserve handles here! */ + Status = NtAllocateVirtualMemory(NtCurrentProcess(), + (PVOID*)&ArrayPointer, + 0, + &ArraySize, + MEM_RESERVE | MEM_COMMIT, + PAGE_READWRITE); + if (!NT_SUCCESS(Status)) + return NULL; + + /* update handle array pointers */ + HandleTable->FreeHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; + HandleTable->MaxReservedHandles = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)ArrayPointer + ArraySize); + HandleTable->CommittedHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; + HandleTable->UnCommittedHandles = (PRTL_HANDLE_TABLE_ENTRY)ArrayPointer; + } /* FIXME - should check if handles need to be committed */ - /* build free list in handle array */ - ph = HandleTable->FreeHandles; - pph = pp_new; - while (ph < HandleTable->MaxReservedHandles) - { - *pph = ph; - pph = &ph->NextFree; - ph = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)ph + HandleTable->SizeOfHandleTableEntry); - } - *pph = 0; - } - - /* remove handle from free list */ - retval = *pp_new; - *pp_new = retval->NextFree; - retval->NextFree = NULL; - - if (Index) - *Index = ((ULONG)((ULONG_PTR)retval - (ULONG_PTR)HandleTable->CommittedHandles) / - HandleTable->SizeOfHandleTableEntry); - - return retval; + /* build free list in handle array */ + ph = HandleTable->FreeHandles; + pph = pp_new; + while (ph < HandleTable->MaxReservedHandles) + { + *pph = ph; + pph = &ph->NextFree; + ph = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)ph + HandleTable->SizeOfHandleTableEntry); + } + *pph = 0; + } + + /* remove handle from free list */ + retval = *pp_new; + *pp_new = retval->NextFree; + retval->NextFree = NULL; + + if (Index) + { + *Index = ((ULONG)((ULONG_PTR)retval - (ULONG_PTR)HandleTable->CommittedHandles) / + HandleTable->SizeOfHandleTableEntry); + } + + return retval; } /* * @implemented */ -BOOLEAN NTAPI -RtlFreeHandle(PRTL_HANDLE_TABLE HandleTable, - PRTL_HANDLE_TABLE_ENTRY Handle) +BOOLEAN +NTAPI +RtlFreeHandle( + PRTL_HANDLE_TABLE HandleTable, + PRTL_HANDLE_TABLE_ENTRY Handle) { #if DBG - /* check if handle is valid */ - if (!RtlIsValidHandle(HandleTable, Handle)) - { - DPRINT1("Invalid Handle! HandleTable=0x%p, Handle=0x%p, Handle->Flags=0x%x\n", - HandleTable, Handle, Handle ? Handle->Flags : 0); - return FALSE; - } + /* check if handle is valid */ + if (!RtlIsValidHandle(HandleTable, Handle)) + { + DPRINT1("Invalid Handle! HandleTable=0x%p, Handle=0x%p, Handle->Flags=0x%x\n", + HandleTable, Handle, Handle ? Handle->Flags : 0); + return FALSE; + } #endif - /* clear handle */ - memset(Handle, 0, HandleTable->SizeOfHandleTableEntry); + /* clear handle */ + memset(Handle, 0, HandleTable->SizeOfHandleTableEntry); - /* add handle to free list */ - Handle->NextFree = HandleTable->FreeHandles; - HandleTable->FreeHandles = Handle; + /* add handle to free list */ + Handle->NextFree = HandleTable->FreeHandles; + HandleTable->FreeHandles = Handle; - return TRUE; + return TRUE; } /* * @implemented */ -BOOLEAN NTAPI -RtlIsValidHandle(PRTL_HANDLE_TABLE HandleTable, - PRTL_HANDLE_TABLE_ENTRY Handle) +BOOLEAN +NTAPI +RtlIsValidHandle( + PRTL_HANDLE_TABLE HandleTable, + PRTL_HANDLE_TABLE_ENTRY Handle) { - if ((HandleTable != NULL) - && (Handle >= HandleTable->CommittedHandles) - && (Handle < HandleTable->MaxReservedHandles) - && (Handle->Flags & RTL_HANDLE_VALID)) - return TRUE; - return FALSE; + if ((HandleTable != NULL) + && (Handle >= HandleTable->CommittedHandles) + && (Handle < HandleTable->MaxReservedHandles) + && (Handle->Flags & RTL_HANDLE_VALID)) + { + return TRUE; + } + return FALSE; } /* * @implemented */ -BOOLEAN NTAPI -RtlIsValidIndexHandle(IN PRTL_HANDLE_TABLE HandleTable, - IN ULONG Index, - OUT PRTL_HANDLE_TABLE_ENTRY *Handle) +BOOLEAN +NTAPI +RtlIsValidIndexHandle( + IN PRTL_HANDLE_TABLE HandleTable, + IN ULONG Index, + OUT PRTL_HANDLE_TABLE_ENTRY *Handle) { - PRTL_HANDLE_TABLE_ENTRY InternalHandle; + PRTL_HANDLE_TABLE_ENTRY InternalHandle; - DPRINT("RtlIsValidIndexHandle(HandleTable %p Index 0x%lx Handle %p)\n", HandleTable, Index, Handle); + DPRINT("RtlIsValidIndexHandle(HandleTable %p Index 0x%lx Handle %p)\n", HandleTable, Index, Handle); - if (HandleTable == NULL) - return FALSE; + if (HandleTable == NULL) + return FALSE; - DPRINT("Handles %p HandleSize 0x%lx\n", - HandleTable->CommittedHandles, HandleTable->SizeOfHandleTableEntry); + DPRINT("Handles %p HandleSize 0x%lx\n", + HandleTable->CommittedHandles, HandleTable->SizeOfHandleTableEntry); - InternalHandle = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)HandleTable->CommittedHandles + - (HandleTable->SizeOfHandleTableEntry * Index)); - if (!RtlIsValidHandle(HandleTable, InternalHandle)) - return FALSE; + InternalHandle = (PRTL_HANDLE_TABLE_ENTRY)((ULONG_PTR)HandleTable->CommittedHandles + + (HandleTable->SizeOfHandleTableEntry * Index)); + if (!RtlIsValidHandle(HandleTable, InternalHandle)) + return FALSE; - DPRINT("InternalHandle %p\n", InternalHandle); + DPRINT("InternalHandle %p\n", InternalHandle); - if (Handle != NULL) - *Handle = InternalHandle; + if (Handle != NULL) + *Handle = InternalHandle; - return TRUE; + return TRUE; } /* EOF */ -- 2.17.1