Ros-diffs

ros-diffs@reactos.org

81390 discussions

[akhaldi] 59598: [CMAKE/GCC] * Compile debug builds using -O1 instead of -Os. This results in faster compilation time with no measurable difference in runtime performance according to tests. CORE-7...
by akhaldi＠svn.reactos.org 31 Jul '13

31 Jul '13

Author: akhaldi Date: Wed Jul 31 13:04:38 2013 New Revision: 59598 URL: http://svn.reactos.org/svn/reactos?rev=59598&view=rev Log: [CMAKE/GCC] * Compile debug builds using -O1 instead of -Os. This results in faster compilation time with no measurable difference in runtime performance according to tests. CORE-7374 #resolve #comment Committed in r59598. Modified: trunk/reactos/cmake/config.cmake Modified: trunk/reactos/cmake/config.cmake URL: http://svn.reactos.org/svn/reactos/trunk/reactos/cmake/config.cmake?rev=595… ============================================================================== --- trunk/reactos/cmake/config.cmake [iso-8859-1] (original) +++ trunk/reactos/cmake/config.cmake [iso-8859-1] Wed Jul 31 13:04:38 2013 @@ -10,7 +10,7 @@ set(TUNE "i686" CACHE STRING "Which CPU ReactOS should be optimized for.") -set(OPTIMIZE "1" CACHE STRING +set(OPTIMIZE "4" CACHE STRING "What level of optimization to use. 0 = off 1 = Default option, optimize for size (-Os) with some additional options

1 0

[akhaldi] 59597: [KERNEL32] * Fix GCC -O0 build.
by akhaldi＠svn.reactos.org 30 Jul '13

30 Jul '13

Author: akhaldi Date: Tue Jul 30 16:13:40 2013 New Revision: 59597 URL: http://svn.reactos.org/svn/reactos?rev=59597&view=rev Log: [KERNEL32] * Fix GCC -O0 build. Modified: trunk/reactos/dll/win32/kernel32/client/file/find.c Modified: trunk/reactos/dll/win32/kernel32/client/file/find.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/kernel32/client/… ============================================================================== --- trunk/reactos/dll/win32/kernel32/client/file/find.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/kernel32/client/file/find.c [iso-8859-1] Tue Jul 30 16:13:40 2013 @@ -547,7 +547,6 @@ } RtlFreeHeap(RtlGetProcessHeap(), 0, FindDataHandle); - _SEH2_YIELD(return TRUE); } _SEH2_EXCEPT(EXCEPTION_EXECUTE_HANDLER) { @@ -555,6 +554,7 @@ _SEH2_YIELD(return FALSE); } _SEH2_END; + return TRUE; }

1 0

[akhaldi] 59596: [PSDK] * Make WS2TCPIP_INLINE and MSTCPIP_INLINE static inline instead of extern inline. This way we don't run into any warnings.
by akhaldi＠svn.reactos.org 30 Jul '13

30 Jul '13

Author: akhaldi Date: Tue Jul 30 16:12:18 2013 New Revision: 59596 URL: http://svn.reactos.org/svn/reactos?rev=59596&view=rev Log: [PSDK] * Make WS2TCPIP_INLINE and MSTCPIP_INLINE static inline instead of extern inline. This way we don't run into any warnings. Modified: trunk/reactos/include/psdk/mstcpip.h trunk/reactos/include/psdk/ws2ipdef.h trunk/reactos/include/psdk/ws2tcpip.h Modified: trunk/reactos/include/psdk/mstcpip.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/mstcpip.h?rev… ============================================================================== --- trunk/reactos/include/psdk/mstcpip.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/mstcpip.h [iso-8859-1] Tue Jul 30 16:12:18 2013 @@ -12,7 +12,7 @@ #ifdef _MSC_VER #define MSTCPIP_INLINE __inline #else -#define MSTCPIP_INLINE extern inline +#define MSTCPIP_INLINE static inline #endif #include <nldef.h> Modified: trunk/reactos/include/psdk/ws2ipdef.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/ws2ipdef.h?re… ============================================================================== --- trunk/reactos/include/psdk/ws2ipdef.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/ws2ipdef.h [iso-8859-1] Tue Jul 30 16:12:18 2013 @@ -13,7 +13,7 @@ #ifdef _MSC_VER #define WS2TCPIP_INLINE __inline #else -#define WS2TCPIP_INLINE extern inline +#define WS2TCPIP_INLINE static inline #endif #include <in6addr.h> Modified: trunk/reactos/include/psdk/ws2tcpip.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/psdk/ws2tcpip.h?re… ============================================================================== --- trunk/reactos/include/psdk/ws2tcpip.h [iso-8859-1] (original) +++ trunk/reactos/include/psdk/ws2tcpip.h [iso-8859-1] Tue Jul 30 16:12:18 2013 @@ -24,7 +24,7 @@ #ifdef _MSC_VER #define WS2TCPIP_INLINE __inline #else -#define WS2TCPIP_INLINE extern inline +#define WS2TCPIP_INLINE static inline #endif /* getaddrinfo error codes */

1 0

[akhaldi] 59595: [INCLUDE/WINE] * Leverage __builtin_ffs() to fix the ffs() related issues.
by akhaldi＠svn.reactos.org 30 Jul '13

30 Jul '13

Author: akhaldi Date: Tue Jul 30 16:06:38 2013 New Revision: 59595 URL: http://svn.reactos.org/svn/reactos?rev=59595&view=rev Log: [INCLUDE/WINE] * Leverage __builtin_ffs() to fix the ffs() related issues. Modified: trunk/reactos/include/reactos/wine/port.h Modified: trunk/reactos/include/reactos/wine/port.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/wine/port.… ============================================================================== --- trunk/reactos/include/reactos/wine/port.h [iso-8859-1] (original) +++ trunk/reactos/include/reactos/wine/port.h [iso-8859-1] Tue Jul 30 16:06:38 2013 @@ -354,6 +354,8 @@ if (_BitScanForward(&index, mask) == 0) return 0; return index; } +#else +#define ffs __builtin_ffs #endif #else /* NO_LIBWINE_PORT */

1 0

[ekohl] 59594: [SAMSRV] Use a common routine to check all account names.
by ekohl＠svn.reactos.org 28 Jul '13

28 Jul '13

Author: ekohl Date: Sun Jul 28 22:18:42 2013 New Revision: 59594 URL: http://svn.reactos.org/svn/reactos?rev=59594&view=rev Log: [SAMSRV] Use a common routine to check all account names. Modified: trunk/reactos/dll/win32/samsrv/samrpc.c trunk/reactos/dll/win32/samsrv/samsrv.h trunk/reactos/dll/win32/samsrv/utils.c Modified: trunk/reactos/dll/win32/samsrv/samrpc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/samsrv/samrpc.c?… ============================================================================== --- trunk/reactos/dll/win32/samsrv/samrpc.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/samsrv/samrpc.c [iso-8859-1] Sun Jul 28 22:18:42 2013 @@ -1756,6 +1756,14 @@ return Status; } + /* Check the group account name */ + Status = SampCheckAccountName(Name, 256); + if (!NT_SUCCESS(Status)) + { + TRACE("SampCheckAccountName failed (Status 0x%08lx)\n", Status); + return Status; + } + /* Check if the group name already exists in the domain */ Status = SampCheckAccountNameInDomain(DomainObject, Name->Buffer); @@ -2128,10 +2136,12 @@ return Status; } - if (Name->Length > 20 * sizeof(WCHAR)) - { - TRACE("User name is too long!\n"); - return STATUS_INVALID_ACCOUNT_NAME; + /* Check the user account name */ + Status = SampCheckAccountName(Name, 20); + if (!NT_SUCCESS(Status)) + { + TRACE("SampCheckAccountName failed (Status 0x%08lx)\n", Status); + return Status; } /* Check if the user name already exists in the domain */ @@ -2691,6 +2701,14 @@ return Status; } + /* Check the alias acoount name */ + Status = SampCheckAccountName(AccountName, 256); + if (!NT_SUCCESS(Status)) + { + TRACE("SampCheckAccountName failed (Status 0x%08lx)\n", Status); + return Status; + } + /* Check if the alias name already exists in the domain */ Status = SampCheckAccountNameInDomain(DomainObject, AccountName->Buffer); @@ -3985,6 +4003,14 @@ goto done; } + /* Check the new account name */ + Status = SampCheckAccountName(&Buffer->Name.Name, 256); + if (!NT_SUCCESS(Status)) + { + TRACE("SampCheckAccountName failed (Status 0x%08lx)\n", Status); + return Status; + } + NewGroupName.Length = Buffer->Name.Name.Length; NewGroupName.MaximumLength = Buffer->Name.Name.MaximumLength; NewGroupName.Buffer = Buffer->Name.Name.Buffer; @@ -4710,6 +4736,14 @@ { TRACE("SampGetObjectAttributeString failed (Status 0x%08lx)\n", Status); goto done; + } + + /* Check the new account name */ + Status = SampCheckAccountName(&Buffer->Name.Name, 256); + if (!NT_SUCCESS(Status)) + { + TRACE("SampCheckAccountName failed (Status 0x%08lx)\n", Status); + return Status; } NewAliasName.Length = Buffer->Name.Name.Length; @@ -6763,6 +6797,14 @@ { UNICODE_STRING OldUserName = {0, 0, NULL}; NTSTATUS Status; + + /* Check the account name */ + Status = SampCheckAccountName(NewUserName, 20); + if (!NT_SUCCESS(Status)) + { + TRACE("SampCheckAccountName failed (Status 0x%08lx)\n", Status); + return Status; + } Status = SampGetObjectAttributeString(UserObject, L"Name", @@ -8097,10 +8139,12 @@ return Status; } - if (Name->Length > 20 * sizeof(WCHAR)) - { - TRACE("User name is too long!\n"); - return STATUS_INVALID_ACCOUNT_NAME; + /* Check the user account name */ + Status = SampCheckAccountName(Name, 20); + if (!NT_SUCCESS(Status)) + { + TRACE("SampCheckAccountName failed (Status 0x%08lx)\n", Status); + return Status; } /* Check if the user name already exists in the domain */ Modified: trunk/reactos/dll/win32/samsrv/samsrv.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/samsrv/samsrv.h?… ============================================================================== --- trunk/reactos/dll/win32/samsrv/samsrv.h [iso-8859-1] (original) +++ trunk/reactos/dll/win32/samsrv/samsrv.h [iso-8859-1] Sun Jul 28 22:18:42 2013 @@ -388,6 +388,10 @@ SampGetRidFromSid(IN PSID Sid, OUT PULONG Rid); +NTSTATUS +SampCheckAccountName(IN PRPC_UNICODE_STRING AccountName, + IN USHORT MaxLength); + /* Undocumented advapi32 functions */ Modified: trunk/reactos/dll/win32/samsrv/utils.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/samsrv/utils.c?r… ============================================================================== --- trunk/reactos/dll/win32/samsrv/utils.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/samsrv/utils.c [iso-8859-1] Sun Jul 28 22:18:42 2013 @@ -155,4 +155,15 @@ return STATUS_SUCCESS; } + +NTSTATUS +SampCheckAccountName(IN PRPC_UNICODE_STRING AccountName, + IN USHORT MaxLength) +{ + if (AccountName->Length > MaxLength * sizeof(WCHAR)) + return STATUS_INVALID_ACCOUNT_NAME; + + return STATUS_SUCCESS; +} + /* EOF */

1 0

[akhaldi] 59593: [RSYM] * Speedup FindOrAddString(). Brought to you by Art Yerkes (arty).
by akhaldi＠svn.reactos.org 28 Jul '13

28 Jul '13

Author: akhaldi Date: Sun Jul 28 20:37:51 2013 New Revision: 59593 URL: http://svn.reactos.org/svn/reactos?rev=59593&view=rev Log: [RSYM] * Speedup FindOrAddString(). Brought to you by Art Yerkes (arty). Modified: trunk/reactos/tools/rsym/rsym.c Modified: trunk/reactos/tools/rsym/rsym.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/rsym/rsym.c?rev=5959… ============================================================================== --- trunk/reactos/tools/rsym/rsym.c [iso-8859-1] (original) +++ trunk/reactos/tools/rsym/rsym.c [iso-8859-1] Sun Jul 28 20:37:51 2013 @@ -33,6 +33,83 @@ #define MAX_PATH 260 #define MAX_SYM_NAME 2000 +struct StringEntry +{ + struct StringEntry *Next; + ULONG Offset; + char *String; +}; + +struct StringHashTable +{ + ULONG TableSize; + struct StringEntry **Table; +}; + +/* This is the famous DJB hash */ +static unsigned int +ComputeDJBHash(const char *name) +{ + unsigned int val = 5381; + int i = 0; + + for (i = 0; name[i]; i++) + { + val = (33 * val) + name[i]; + } + + return val; +} + +static void +AddStringToHash(struct StringHashTable *StringTable, + unsigned int hash, + ULONG Offset, + char *StringPtr) +{ + struct StringEntry *entry = calloc(1, sizeof(struct StringEntry)); + entry->Offset = Offset; + entry->String = StringPtr; + entry->Next = StringTable->Table[hash]; + StringTable->Table[hash] = entry; +} + +static void +StringHashTableInit(struct StringHashTable *StringTable, + ULONG StringsLength, + char *StringsBase) +{ + char *Start = StringsBase; + char *End = StringsBase + StringsLength; + StringTable->TableSize = 1024; + StringTable->Table = calloc(1024, sizeof(struct StringEntry *)); + while (Start < End) + { + AddStringToHash(StringTable, + ComputeDJBHash(Start) % StringTable->TableSize, + Start - StringsBase, + Start); + Start += strlen(Start) + 1; + } +} + +static void +StringHashTableFree(struct StringHashTable *StringTable) +{ + int i; + struct StringEntry *entry; + for (i = 0; i < StringTable->TableSize; i++) + { + while ((entry = StringTable->Table[i])) + { + entry = entry->Next; + free(StringTable->Table[i]); + StringTable->Table[i] = entry; + } + } + free(StringTable->Table); +} + static int CompareSymEntry(const PROSSYM_ENTRY SymEntry1, const PROSSYM_ENTRY SymEntry2) { @@ -112,26 +189,32 @@ } static ULONG -FindOrAddString(char *StringToFind, ULONG *StringsLength, void *StringsBase) -{ - char *Search, *End; - - Search = (char *)StringsBase; - End = Search + *StringsLength; - - while (Search < End) - { - if (strcmp(Search, StringToFind) == 0) - { - return Search - (char *)StringsBase; - } - Search += strlen(Search) + 1; - } - - strcpy(Search, StringToFind); - *StringsLength += strlen(StringToFind) + 1; - - return Search - (char *)StringsBase; +FindOrAddString(struct StringHashTable *StringTable, + char *StringToFind, + ULONG *StringsLength, + void *StringsBase) +{ + unsigned int hash = ComputeDJBHash(StringToFind) % StringTable->TableSize; + struct StringEntry *entry = StringTable->Table[hash]; + + while (entry && strcmp(entry->String, StringToFind)) + entry = entry->Next; + + if (entry) + { + return entry->Offset; + } + else + { + char *End = (char *)StringsBase + *StringsLength; + + strcpy(End, StringToFind); + *StringsLength += strlen(StringToFind) + 1; + + AddStringToHash(StringTable, hash, End - (char *)StringsBase, End); + + return End - (char *)StringsBase; + } } static int @@ -150,6 +233,7 @@ ULONG NameLen; char FuncName[256]; PROSSYM_ENTRY Current; + struct StringHashTable StringHash; StabEntry = StabSymbolsBase; Count = StabSymbolsLength / sizeof(STAB_ENTRY); @@ -170,6 +254,8 @@ } Current = *SymbolsBase; memset(Current, 0, sizeof(*Current)); + + StringHashTableInit(&StringHash, *StringsLength, (char *)StringsBase); LastFunctionAddress = 0; for (i = 0; i < Count; i++) @@ -206,8 +292,8 @@ First = 0; Current->Address = Address; } - Current->FileOffset = FindOrAddString((char *)StabStringsBase - + StabEntry[i].n_strx, + Current->FileOffset = FindOrAddString(&StringHash, + (char *)StabStringsBase + StabEntry[i].n_strx, StringsLength, StringsBase); break; @@ -236,7 +322,8 @@ } memcpy(FuncName, Name, NameLen); FuncName[NameLen] = '\0'; - Current->FunctionOffset = FindOrAddString(FuncName, + Current->FunctionOffset = FindOrAddString(&StringHash, + FuncName, StringsLength, StringsBase); Current->SourceLine = 0; @@ -265,6 +352,8 @@ qsort(*SymbolsBase, *SymbolsCount, sizeof(ROSSYM_ENTRY), (int (*)(const void *, const void *)) CompareSymEntry); + StringHashTableFree(&StringHash); + return 0; } @@ -281,6 +370,7 @@ char FuncName[256], FileName[1024]; char *p; PROSSYM_ENTRY Current; + struct StringHashTable StringHash; CoffEntry = (PCOFF_SYMENT) CoffSymbolsBase; Count = CoffSymbolsLength / sizeof(COFF_SYMENT); @@ -293,6 +383,8 @@ } *SymbolsCount = 0; Current = *SymbolsBase; + + StringHashTableInit(&StringHash, *StringsLength, (char*)StringsBase); for (i = 0; i < Count; i++) { @@ -319,6 +411,7 @@ { free(*SymbolsBase); fprintf(stderr, "Function name too long\n"); + StringHashTableFree(&StringHash); return 1; } strcpy(FuncName, (char *) CoffStringsBase + CoffEntry[i].e.e.e_offset); @@ -336,7 +429,8 @@ *p = '\0'; } p = ('_' == FuncName[0] || '@' == FuncName[0] ? FuncName + 1 : FuncName); - Current->FunctionOffset = FindOrAddString(p, + Current->FunctionOffset = FindOrAddString(&StringHash, + p, StringsLength, StringsBase); Current->SourceLine = 0; @@ -348,6 +442,8 @@ *SymbolsCount = (Current - *SymbolsBase + 1); qsort(*SymbolsBase, *SymbolsCount, sizeof(ROSSYM_ENTRY), (int (*)(const void *, const void *)) CompareSymEntry); + + StringHashTableFree(&StringHash); return 0; } @@ -370,21 +466,6 @@ DWORD module_base; struct DbgHelpLineEntry *lastLineEntry; }; - -/* This is the famous DJB hash */ -static unsigned int -ComputeDJBHash(const char *name) -{ - unsigned int val = 5381; - int i = 0; - - for (i = 0; name[i]; i++) - { - val = (33 * val) + name[i]; - } - - return val; -} static struct DbgHelpLineEntry* DbgHelpAddLineEntry(struct DbgHelpStringTab *tab)

1 0

[tkreuzer] 59592: [CSRSRV] Use symbolic constant for number of hash buckets and use it where appropriate (fixing one missing location).
by tkreuzer＠svn.reactos.org 28 Jul '13

28 Jul '13

Author: tkreuzer Date: Sun Jul 28 13:54:42 2013 New Revision: 59592 URL: http://svn.reactos.org/svn/reactos?rev=59592&view=rev Log: [CSRSRV] Use symbolic constant for number of hash buckets and use it where appropriate (fixing one missing location). Modified: trunk/reactos/subsystems/win32/csrsrv/api.h trunk/reactos/subsystems/win32/csrsrv/procsup.c trunk/reactos/subsystems/win32/csrsrv/thredsup.c Modified: trunk/reactos/subsystems/win32/csrsrv/api.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/ap… ============================================================================== --- trunk/reactos/subsystems/win32/csrsrv/api.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/csrsrv/api.h [iso-8859-1] Sun Jul 28 13:54:42 2013 @@ -45,7 +45,8 @@ extern HANDLE CsrApiPort; extern HANDLE CsrSmApiPort; extern HANDLE CsrSbApiPort; -extern LIST_ENTRY CsrThreadHashTable[257]; +#define NUMBER_THREAD_HASH_BUCKETS 257 +extern LIST_ENTRY CsrThreadHashTable[NUMBER_THREAD_HASH_BUCKETS]; extern PCSR_PROCESS CsrRootProcess; extern UNICODE_STRING CsrDirectoryName; extern ULONG CsrDebug; Modified: trunk/reactos/subsystems/win32/csrsrv/procsup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/pr… ============================================================================== --- trunk/reactos/subsystems/win32/csrsrv/procsup.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/csrsrv/procsup.c [iso-8859-1] Sun Jul 28 13:54:42 2013 @@ -371,7 +371,7 @@ CsrRootProcess->ClientId = NtCurrentTeb()->ClientId; /* Initialize the Thread Hash List */ - for (i = 0; i < 256; i++) InitializeListHead(&CsrThreadHashTable[i]); + for (i = 0; i < NUMBER_THREAD_HASH_BUCKETS; i++) InitializeListHead(&CsrThreadHashTable[i]); /* Initialize the Wait Lock */ return RtlInitializeCriticalSection(&CsrWaitListsLock); @@ -1382,7 +1382,7 @@ /* Acquire process lock */ CsrAcquireProcessLock(); - + /* Get the list pointers */ NextEntry = CsrRootProcess->ListLink.Flink; while (NextEntry != &CsrRootProcess->ListLink) @@ -1397,13 +1397,13 @@ CsrProcess->Flags &= ~CsrProcessSkipShutdown; CsrProcess->ShutdownFlags = 0; } - + /* Set shudown Priority */ CsrSetToShutdownPriority(); /* Loop all processes */ //DPRINT1("Enumerating for LUID: %lx %lx\n", CallerLuid->HighPart, CallerLuid->LowPart); - + /* Start looping */ while (TRUE) { Modified: trunk/reactos/subsystems/win32/csrsrv/thredsup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/th… ============================================================================== --- trunk/reactos/subsystems/win32/csrsrv/thredsup.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/csrsrv/thredsup.c [iso-8859-1] Sun Jul 28 13:54:42 2013 @@ -14,11 +14,11 @@ #define NDEBUG #include <debug.h> -#define CsrHashThread(t) (HandleToUlong(t) % 257) +#define CsrHashThread(t) (HandleToUlong(t) % NUMBER_THREAD_HASH_BUCKETS) /* GLOBALS ********************************************************************/ -LIST_ENTRY CsrThreadHashTable[257]; +LIST_ENTRY CsrThreadHashTable[NUMBER_THREAD_HASH_BUCKETS]; /* PRIVATE FUNCTIONS **********************************************************/

1 0

[tkreuzer] 59591: [CSRSRV] - Reduce number of hash collisions during bootup from 10 to 0, by choosing 257 (prime number = good) instead of 256 (power of 2 = bad) - Use ULONG for CsrpStaticThreadCou...
by tkreuzer＠svn.reactos.org 28 Jul '13

28 Jul '13

Author: tkreuzer Date: Sun Jul 28 10:41:27 2013 New Revision: 59591 URL: http://svn.reactos.org/svn/reactos?rev=59591&view=rev Log: [CSRSRV] - Reduce number of hash collisions during bootup from 10 to 0, by choosing 257 (prime number = good) instead of 256 (power of 2 = bad) - Use ULONG for CsrpStaticThreadCount and CsrpDynamicThreadTotal to fix an MSVC warning. Modified: trunk/reactos/subsystems/win32/csrsrv/api.c trunk/reactos/subsystems/win32/csrsrv/api.h trunk/reactos/subsystems/win32/csrsrv/srv.h trunk/reactos/subsystems/win32/csrsrv/thredsup.c Modified: trunk/reactos/subsystems/win32/csrsrv/api.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/ap… ============================================================================== --- trunk/reactos/subsystems/win32/csrsrv/api.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/csrsrv/api.c [iso-8859-1] Sun Jul 28 10:41:27 2013 @@ -18,8 +18,8 @@ BOOLEAN (*CsrClientThreadSetup)(VOID) = NULL; UNICODE_STRING CsrApiPortName; -volatile LONG CsrpStaticThreadCount; -volatile LONG CsrpDynamicThreadTotal; +volatile ULONG CsrpStaticThreadCount; +volatile ULONG CsrpDynamicThreadTotal; extern ULONG CsrMaxApiRequestThreads; /* FUNCTIONS ******************************************************************/ @@ -269,7 +269,7 @@ NTSTATUS Status; /* Decrease the count, and see if we're out */ - if (_InterlockedDecrement(&CsrpStaticThreadCount) == 0) + if (InterlockedDecrementUL(&CsrpStaticThreadCount) == 0) { /* Check if we've still got space for a Dynamic Thread */ if (CsrpDynamicThreadTotal < CsrMaxApiRequestThreads) @@ -289,8 +289,8 @@ if (NT_SUCCESS(Status)) { /* Increase the thread counts */ - _InterlockedIncrement(&CsrpStaticThreadCount); - _InterlockedIncrement(&CsrpDynamicThreadTotal); + InterlockedIncrementUL(&CsrpStaticThreadCount); + InterlockedIncrementUL(&CsrpDynamicThreadTotal); /* Add a new server thread */ if (CsrAddStaticServerThread(hThread, @@ -303,8 +303,8 @@ else { /* Failed to create a new static thread */ - _InterlockedDecrement(&CsrpStaticThreadCount); - _InterlockedDecrement(&CsrpDynamicThreadTotal); + InterlockedDecrementUL(&CsrpStaticThreadCount); + InterlockedDecrementUL(&CsrpDynamicThreadTotal); /* Terminate it */ DPRINT1("Failing\n"); @@ -383,8 +383,8 @@ ASSERT(NT_SUCCESS(Status)); /* Increase the Thread Counts */ - _InterlockedIncrement(&CsrpStaticThreadCount); - _InterlockedIncrement(&CsrpDynamicThreadTotal); + InterlockedIncrementUL(&CsrpStaticThreadCount); + InterlockedIncrementUL(&CsrpDynamicThreadTotal); } /* Now start the loop */ @@ -513,7 +513,7 @@ } /* Increase the thread count */ - _InterlockedIncrement(&CsrpStaticThreadCount); + InterlockedIncrementUL(&CsrpStaticThreadCount); /* If the response was 0xFFFFFFFF, we'll ignore it */ if (HardErrorMsg->Response == 0xFFFFFFFF) @@ -595,7 +595,7 @@ ServerDll->DispatchTable[ApiId](&ReceiveMsg, &ReplyCode); /* Increase the static thread count */ - _InterlockedIncrement(&CsrpStaticThreadCount); + InterlockedIncrementUL(&CsrpStaticThreadCount); } _SEH2_EXCEPT(CsrUnhandledExceptionFilter(_SEH2_GetExceptionInformation())) { @@ -706,7 +706,7 @@ } /* Increase the thread count */ - _InterlockedIncrement(&CsrpStaticThreadCount); + InterlockedIncrementUL(&CsrpStaticThreadCount); /* If the response was 0xFFFFFFFF, we'll ignore it */ if (HardErrorMsg->Response == 0xFFFFFFFF) @@ -818,7 +818,7 @@ ReplyMsg->Status = ServerDll->DispatchTable[ApiId](&ReceiveMsg, &ReplyCode); /* Increase the static thread count */ - _InterlockedIncrement(&CsrpStaticThreadCount); + InterlockedIncrementUL(&CsrpStaticThreadCount); Teb->CsrClientThread = CurrentThread; Modified: trunk/reactos/subsystems/win32/csrsrv/api.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/ap… ============================================================================== --- trunk/reactos/subsystems/win32/csrsrv/api.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/csrsrv/api.h [iso-8859-1] Sun Jul 28 10:41:27 2013 @@ -45,7 +45,7 @@ extern HANDLE CsrApiPort; extern HANDLE CsrSmApiPort; extern HANDLE CsrSbApiPort; -extern LIST_ENTRY CsrThreadHashTable[256]; +extern LIST_ENTRY CsrThreadHashTable[257]; extern PCSR_PROCESS CsrRootProcess; extern UNICODE_STRING CsrDirectoryName; extern ULONG CsrDebug; Modified: trunk/reactos/subsystems/win32/csrsrv/srv.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/sr… ============================================================================== --- trunk/reactos/subsystems/win32/csrsrv/srv.h [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/csrsrv/srv.h [iso-8859-1] Sun Jul 28 10:41:27 2013 @@ -55,6 +55,8 @@ #define ROUND_UP(n, align) ROUND_DOWN(((ULONG)n) + (align) - 1, (align)) #define ROUND_DOWN(n, align) (((ULONG)n) & ~((align) - 1l)) +#define InterlockedIncrementUL(Value) _InterlockedIncrement((PLONG)Value) +#define InterlockedDecrementUL(Value) _InterlockedDecrement((PLONG)Value) #endif // _SRV_H Modified: trunk/reactos/subsystems/win32/csrsrv/thredsup.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/subsystems/win32/csrsrv/th… ============================================================================== --- trunk/reactos/subsystems/win32/csrsrv/thredsup.c [iso-8859-1] (original) +++ trunk/reactos/subsystems/win32/csrsrv/thredsup.c [iso-8859-1] Sun Jul 28 10:41:27 2013 @@ -14,11 +14,11 @@ #define NDEBUG #include <debug.h> -#define CsrHashThread(t) (HandleToUlong(t)&(256 - 1)) +#define CsrHashThread(t) (HandleToUlong(t) % 257) /* GLOBALS ********************************************************************/ -LIST_ENTRY CsrThreadHashTable[256]; +LIST_ENTRY CsrThreadHashTable[257]; /* PRIVATE FUNCTIONS **********************************************************/ @@ -581,7 +581,7 @@ DPRINT1("No known process for %lx\n", ClientId->UniqueProcess); return Status; } - + /* Make sure the thread didn't terminate */ if (KernelTimes.ExitTime.QuadPart) {

1 0

[ekohl] 59590: [NETAPI32] NetpNtStatusToApiStatus: Map STATUS_INVALID_ACCOUNT_NAME to NERR_BadUsername.
by ekohl＠svn.reactos.org 27 Jul '13

27 Jul '13

Author: ekohl Date: Sat Jul 27 18:53:52 2013 New Revision: 59590 URL: http://svn.reactos.org/svn/reactos?rev=59590&view=rev Log: [NETAPI32] NetpNtStatusToApiStatus: Map STATUS_INVALID_ACCOUNT_NAME to NERR_BadUsername. Modified: trunk/reactos/dll/win32/netapi32/netapi32.c Modified: trunk/reactos/dll/win32/netapi32/netapi32.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/netapi32/netapi3… ============================================================================== --- trunk/reactos/dll/win32/netapi32/netapi32.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/netapi32/netapi32.c [iso-8859-1] Sat Jul 27 18:53:52 2013 @@ -218,7 +218,24 @@ WINAPI NetpNtStatusToApiStatus(NTSTATUS Status) { - return RtlNtStatusToDosError(Status); + NET_API_STATUS ApiStatus; + + switch (Status) + { + case STATUS_SUCCESS: + ApiStatus = NERR_Success; + break; + + case STATUS_INVALID_ACCOUNT_NAME: + ApiStatus = NERR_BadUsername; + break; + + default: + ApiStatus = RtlNtStatusToDosError(Status); + break; + } + + return ApiStatus; } NET_API_STATUS WINAPI NetUseEnum(LMSTR server, DWORD level, LPBYTE* bufptr, DWORD prefmaxsize,

1 0

[ekohl] 59589: [SAMSRV] SamrCreateUserInDomain and SamrCreateUser2InDomain: Implement the user name length limit of 20 characters.
by ekohl＠svn.reactos.org 27 Jul '13

27 Jul '13

Author: ekohl Date: Sat Jul 27 17:59:28 2013 New Revision: 59589 URL: http://svn.reactos.org/svn/reactos?rev=59589&view=rev Log: [SAMSRV] SamrCreateUserInDomain and SamrCreateUser2InDomain: Implement the user name length limit of 20 characters. Modified: trunk/reactos/dll/win32/samsrv/samrpc.c Modified: trunk/reactos/dll/win32/samsrv/samrpc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/samsrv/samrpc.c?… ============================================================================== --- trunk/reactos/dll/win32/samsrv/samrpc.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/samsrv/samrpc.c [iso-8859-1] Sat Jul 27 17:59:28 2013 @@ -2126,6 +2126,12 @@ { TRACE("failed with status 0x%08lx\n", Status); return Status; + } + + if (Name->Length > 20 * sizeof(WCHAR)) + { + TRACE("User name is too long!\n"); + return STATUS_INVALID_ACCOUNT_NAME; } /* Check if the user name already exists in the domain */ @@ -8091,6 +8097,12 @@ return Status; } + if (Name->Length > 20 * sizeof(WCHAR)) + { + TRACE("User name is too long!\n"); + return STATUS_INVALID_ACCOUNT_NAME; + } + /* Check if the user name already exists in the domain */ Status = SampCheckAccountNameInDomain(DomainObject, Name->Buffer);

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs