Ros-diffs February 2015

ros-diffs@reactos.org

16 participants
372 discussions

[ekohl] 66307: [NTOSKRNL] NtQueryInformationFile: Implement FileAccessInformation and FileAlignmentInformation cases according to 'File System Internals' page 485.
by ekohl＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: ekohl Date: Sun Feb 15 23:06:30 2015 New Revision: 66307 URL: http://svn.reactos.org/svn/reactos?rev=66307&view=rev Log: [NTOSKRNL] NtQueryInformationFile: Implement FileAccessInformation and FileAlignmentInformation cases according to 'File System Internals' page 485. Modified: trunk/reactos/ntoskrnl/io/iomgr/iofunc.c Modified: trunk/reactos/ntoskrnl/io/iomgr/iofunc.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/io/iomgr/iofunc.c… ============================================================================== --- trunk/reactos/ntoskrnl/io/iomgr/iofunc.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/io/iomgr/iofunc.c [iso-8859-1] Sun Feb 15 23:06:30 2015 @@ -1863,6 +1863,9 @@ PVOID NormalContext; KIRQL OldIrql; IO_STATUS_BLOCK KernelIosb; + BOOLEAN CallDriver = TRUE; + PFILE_ACCESS_INFORMATION AccessBuffer; + PFILE_ALIGNMENT_INFORMATION AlignmentBuffer; PAGED_CODE(); IOTRACE(IO_API_DEBUG, "FileHandle: %p\n", FileHandle); @@ -2043,8 +2046,34 @@ /* Update operation counts */ IopUpdateOperationCount(IopOtherTransfer); + /* Fill in file information before calling the driver. + See 'File System Internals' page 485.*/ + if (FileInformationClass == FileAccessInformation) + { + AccessBuffer = Irp->AssociatedIrp.SystemBuffer; + AccessBuffer->AccessFlags = HandleInformation.GrantedAccess; + Irp->IoStatus.Information = sizeof(FILE_ACCESS_INFORMATION); + CallDriver = FALSE; + } + else if (FileInformationClass == FileAlignmentInformation) + { + AlignmentBuffer = Irp->AssociatedIrp.SystemBuffer; + AlignmentBuffer->AlignmentRequirement = DeviceObject->AlignmentRequirement; + Irp->IoStatus.Information = sizeof(FILE_ALIGNMENT_INFORMATION); + CallDriver = FALSE; + } + /* Call the Driver */ - Status = IoCallDriver(DeviceObject, Irp); + if (CallDriver) + { + Status = IoCallDriver(DeviceObject, Irp); + } + else + { + Status = STATUS_SUCCESS; + Irp->IoStatus.Status = STATUS_SUCCESS; + } + if (Status == STATUS_PENDING) { /* Check if this was async I/O */

1 0

[hbelusca] 66306: [WINSRV] - Part 2/2 of SrvLogon (see r66303): load the per-user shutdown timeouts. - Retrieve those per-user shutdown timeouts from the registry when the user logs on, adapted fro...
by hbelusca＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: hbelusca Date: Sun Feb 15 22:57:40 2015 New Revision: 66306 URL: http://svn.reactos.org/svn/reactos?rev=66306&view=rev Log: [WINSRV] - Part 2/2 of SrvLogon (see r66303): load the per-user shutdown timeouts. - Retrieve those per-user shutdown timeouts from the registry when the user logs on, adapted from Alex' shutdown patch plus existing code. - Commit the main part of shutdown code (finally!), still unfinished yet: * Need to switch to the desktop where the hanging GUI app's window is present, * Need to deal with apps from other users * What about SYSTEM processes? * What about console processes? - Reuse the old timeout code. - Fix the sending of WM_QUERYENDSESSION and WM_ENDSESSION messages (this is done by win32k directly; for that winsrv needs just to send one WM_CLIENTSHUTDOWN message with the correct wParam parameter). Part 13/X CORE-8322 #comment Big commit in revision 66306! Modified: trunk/reactos/win32ss/user/winsrv/consrv/consrv.h trunk/reactos/win32ss/user/winsrv/init.c trunk/reactos/win32ss/user/winsrv/usersrv/init.c trunk/reactos/win32ss/user/winsrv/usersrv/shutdown.c trunk/reactos/win32ss/user/winsrv/winsrv.h Modified: trunk/reactos/win32ss/user/winsrv/consrv/consrv.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/winsrv/consrv… ============================================================================== --- trunk/reactos/win32ss/user/winsrv/consrv/consrv.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/winsrv/consrv/consrv.h [iso-8859-1] Sun Feb 15 22:57:40 2015 @@ -18,7 +18,6 @@ #define COM_NO_WINDOWS_H #include <winnls.h> -#include <winreg.h> #include <wincon.h> #define NTOS_MODE_USER Modified: trunk/reactos/win32ss/user/winsrv/init.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/winsrv/init.c… ============================================================================== --- trunk/reactos/win32ss/user/winsrv/init.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/winsrv/init.c [iso-8859-1] Sun Feb 15 22:57:40 2015 @@ -13,6 +13,157 @@ #define NDEBUG #include <debug.h> + +/* GLOBALS ********************************************************************/ + +#define DEFAULT_AUTO_END_TASKS FALSE +#define DEFAULT_HUNG_APP_TIMEOUT 5000 +#define DEFAULT_WAIT_TO_KILL_APP_TIMEOUT 20000 +#define DEFAULT_PROCESS_TERMINATE_TIMEOUT 90000 + +SHUTDOWN_SETTINGS ShutdownSettings = +{ + DEFAULT_AUTO_END_TASKS, + DEFAULT_HUNG_APP_TIMEOUT, + DEFAULT_WAIT_TO_KILL_APP_TIMEOUT, + DEFAULT_WAIT_TO_KILL_APP_TIMEOUT, + DEFAULT_PROCESS_TERMINATE_TIMEOUT +}; + +/* FUNCTIONS ******************************************************************/ + +static ULONG +GetRegIntFromID(IN HANDLE KeyHandle, + IN PWCHAR ValueName, + IN ULONG DefaultValue) +{ + UNICODE_STRING ValueString; + ULONG Length; + UCHAR Buffer[sizeof(KEY_VALUE_PARTIAL_INFORMATION) + 32 * sizeof(WCHAR)]; + PKEY_VALUE_PARTIAL_INFORMATION PartialInfo = (PVOID)Buffer; + NTSTATUS Status; + ULONG Value; + + /* Open the key */ + RtlInitUnicodeString(&ValueString, ValueName); + Status = NtQueryValueKey(KeyHandle, + &ValueString, + KeyValuePartialInformation, + PartialInfo, + sizeof(Buffer), + &Length); + if (NT_SUCCESS(Status)) + { + if (PartialInfo->Type == REG_SZ) + { + /* Convert to integer */ + RtlInitUnicodeString(&ValueString, (PWCHAR)PartialInfo->Data); + Status = RtlUnicodeStringToInteger(&ValueString, 10, &Value); + } + else if (PartialInfo->Type == REG_DWORD) + { + /* Directly retrieve the data */ + Value = *(PULONG)PartialInfo->Data; + Status = STATUS_SUCCESS; + } + else + { + DPRINT1("Unexpected registry type %d for setting %S\n", PartialInfo->Type, ValueName); + Status = STATUS_UNSUCCESSFUL; + } + } + + if (!NT_SUCCESS(Status)) + { + /* Use default value instead */ + Value = DefaultValue; + } + + /* Return the value */ + return Value; +} + +VOID FASTCALL +GetTimeouts(IN PSHUTDOWN_SETTINGS ShutdownSettings) +{ + NTSTATUS Status; + UNICODE_STRING RegistryString; + OBJECT_ATTRIBUTES ObjectAttributes; + HANDLE CurrentUserKeyHandle; + HANDLE KeyHandle; + + /* Initialize with defaults first */ + ShutdownSettings->AutoEndTasks = DEFAULT_AUTO_END_TASKS; + ShutdownSettings->HungAppTimeout = DEFAULT_HUNG_APP_TIMEOUT; + ShutdownSettings->WaitToKillAppTimeout = DEFAULT_WAIT_TO_KILL_APP_TIMEOUT; + ShutdownSettings->WaitToKillServiceTimeout = ShutdownSettings->WaitToKillAppTimeout; + ShutdownSettings->ProcessTerminateTimeout = DEFAULT_PROCESS_TERMINATE_TIMEOUT; + + /* Open the per-user desktop key */ + Status = RtlOpenCurrentUser(MAXIMUM_ALLOWED, &CurrentUserKeyHandle); + if (NT_SUCCESS(Status)) + { + RtlInitUnicodeString(&RegistryString, + L"Control Panel\\Desktop"); + InitializeObjectAttributes(&ObjectAttributes, + &RegistryString, + OBJ_CASE_INSENSITIVE, + CurrentUserKeyHandle, + NULL); + Status = NtOpenKey(&KeyHandle, KEY_READ, &ObjectAttributes); + if (NT_SUCCESS(Status)) + { + /* Read timeouts */ + ShutdownSettings->HungAppTimeout = GetRegIntFromID(KeyHandle, + L"HungAppTimeout", + DEFAULT_HUNG_APP_TIMEOUT); + ShutdownSettings->WaitToKillAppTimeout = GetRegIntFromID(KeyHandle, + L"WaitToKillAppTimeout", + DEFAULT_WAIT_TO_KILL_APP_TIMEOUT); + ShutdownSettings->AutoEndTasks = GetRegIntFromID(KeyHandle, + L"AutoEndTasks", + DEFAULT_AUTO_END_TASKS); + /* Done */ + NtClose(KeyHandle); + } + + /* Done */ + NtClose(CurrentUserKeyHandle); + } + + /* Now open the control key */ + RtlInitUnicodeString(&RegistryString, + L"\\Registry\\Machine\\System\\CurrentControlSet\\Control"); + InitializeObjectAttributes(&ObjectAttributes, + &RegistryString, + OBJ_CASE_INSENSITIVE, + NULL, NULL); + Status = NtOpenKey(&KeyHandle, KEY_READ, &ObjectAttributes); + if (NT_SUCCESS(Status)) + { + /* Read the services timeout */ + ShutdownSettings->WaitToKillServiceTimeout = GetRegIntFromID(KeyHandle, + L"WaitToKillServiceTimeout", + ShutdownSettings->WaitToKillAppTimeout); + + /* + * Retrieve the process terminate timeout. + * See ftp://ftp.microsoft.com/MISC1/BUSSYS/WINNT/KB/Q234/6/06.TXT + * and https://web.archive.org/web/20050216235758/http://support.microsoft.com/kb/… + * for more details. + * + * NOTE: Unused at the moment... + */ + ShutdownSettings->ProcessTerminateTimeout = GetRegIntFromID(KeyHandle, + L"ProcessTerminateTimeout", + DEFAULT_PROCESS_TERMINATE_TIMEOUT); + if (ShutdownSettings->ProcessTerminateTimeout < DEFAULT_HUNG_APP_TIMEOUT) + ShutdownSettings->ProcessTerminateTimeout = DEFAULT_HUNG_APP_TIMEOUT; + + /* Done */ + NtClose(KeyHandle); + } +} /* ENTRY-POINT ****************************************************************/ Modified: trunk/reactos/win32ss/user/winsrv/usersrv/init.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/winsrv/usersr… ============================================================================== --- trunk/reactos/win32ss/user/winsrv/usersrv/init.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/winsrv/usersrv/init.c [iso-8859-1] Sun Feb 15 22:57:40 2015 @@ -135,6 +135,23 @@ { DPRINT1("%s not yet implemented\n", __FUNCTION__); return STATUS_NOT_IMPLEMENTED; +} + +CSR_API(SrvLogon) +{ + PUSER_LOGON LogonRequest = &((PUSER_API_MESSAGE)ApiMessage)->Data.LogonRequest; + + DPRINT1("We are logged %s\n", LogonRequest->IsLogon ? "on" : "off"); + + /* Impersonate the caller in order to retrieve settings in its context */ + if (!CsrImpersonateClient(NULL)) + return STATUS_UNSUCCESSFUL; + + GetTimeouts(&ShutdownSettings); + + /* We are done */ + CsrRevertToSelf(); + return STATUS_SUCCESS; } NTSTATUS Modified: trunk/reactos/win32ss/user/winsrv/usersrv/shutdown.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/winsrv/usersr… ============================================================================== --- trunk/reactos/win32ss/user/winsrv/usersrv/shutdown.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/winsrv/usersrv/shutdown.c [iso-8859-1] Sun Feb 15 22:57:40 2015 @@ -4,19 +4,14 @@ * FILE: win32ss/user/winsrv/usersrv/shutdown.c * PURPOSE: Logout/shutdown * PROGRAMMERS: - * - * NOTE: The shutdown code must be rewritten completely. (hbelusca) */ /* INCLUDES *******************************************************************/ #include "usersrv.h" -#include <stdlib.h> -#include <winreg.h> #include <winlogon.h> #include <commctrl.h> -#include <sddl.h> #include "resource.h" @@ -25,27 +20,25 @@ /* GLOBALS ********************************************************************/ -typedef struct tagSHUTDOWN_SETTINGS -{ - BOOL AutoEndTasks; - DWORD HungAppTimeout; - DWORD WaitToKillAppTimeout; -} SHUTDOWN_SETTINGS, *PSHUTDOWN_SETTINGS; - -SHUTDOWN_SETTINGS ShutdownSettings = {0}; - -#define DEFAULT_AUTO_END_TASKS FALSE -#define DEFAULT_HUNG_APP_TIMEOUT 5000 -#define DEFAULT_WAIT_TO_KILL_APP_TIMEOUT 20000 +// Those flags (that are used for CsrProcess->ShutdownFlags) are named +// in accordance to the only public one: SHUTDOWN_NORETRY used for the +// SetProcessShutdownParameters API. +#if !defined(SHUTDOWN_SYSTEMCONTEXT) && !defined(SHUTDOWN_OTHERCONTEXT) +#define SHUTDOWN_SYSTEMCONTEXT CsrShutdownSystem +#define SHUTDOWN_OTHERCONTEXT CsrShutdownOther +#endif + +// The DPRINTs that need to really be removed as soon as everything works. +#define MY_DPRINT DPRINT1 +#define MY_DPRINT2 DPRINT typedef struct tagNOTIFY_CONTEXT { - DWORD ProcessId; UINT Msg; WPARAM wParam; LPARAM lParam; - HDESK Desktop; - HDESK OldDesktop; + // HDESK Desktop; + // HDESK OldDesktop; DWORD StartTime; DWORD QueryResult; HWND Dlg; @@ -54,7 +47,6 @@ HANDLE UIThread; HWND WndClient; PSHUTDOWN_SETTINGS ShutdownSettings; - LPTHREAD_START_ROUTINE SendMessageProc; } NOTIFY_CONTEXT, *PNOTIFY_CONTEXT; #define QUERY_RESULT_ABORT 0 @@ -74,17 +66,29 @@ DWORD Timeout; } MESSAGE_CONTEXT, *PMESSAGE_CONTEXT; -typedef struct tagPROCESS_ENUM_CONTEXT -{ - UINT ProcessCount; - PCSR_PROCESS *ProcessData; - TOKEN_ORIGIN TokenOrigin; - DWORD ShellProcess; - DWORD CsrssProcess; -} PROCESS_ENUM_CONTEXT, *PPROCESS_ENUM_CONTEXT; - /* FUNCTIONS ******************************************************************/ + +static HMODULE hComCtl32Lib = NULL; + +static VOID +CallInitCommonControls(VOID) +{ + static BOOL Initialized = FALSE; + INITCOMMONCONTROLS_PROC InitProc; + + if (Initialized) return; + + hComCtl32Lib = LoadLibraryW(L"COMCTL32.DLL"); + if (hComCtl32Lib == NULL) return; + + InitProc = (INITCOMMONCONTROLS_PROC)GetProcAddress(hComCtl32Lib, "InitCommonControls"); + if (InitProc == NULL) return; + + (*InitProc)(); + + Initialized = TRUE; +} static VOID FASTCALL UpdateProgressBar(HWND ProgressBar, PNOTIFY_CONTEXT NotifyContext) @@ -113,18 +117,18 @@ switch(Msg) { case WM_INITDIALOG: - NotifyContext = (PNOTIFY_CONTEXT) lParam; + NotifyContext = (PNOTIFY_CONTEXT)lParam; NotifyContext->EndNowResult = QUERY_RESULT_ABORT; - SetWindowLongPtrW(Dlg, DWLP_USER, (LONG_PTR) lParam); + SetWindowLongPtrW(Dlg, DWLP_USER, (LONG_PTR)lParam); TitleLength = SendMessageW(NotifyContext->WndClient, WM_GETTEXTLENGTH, 0, 0) + GetWindowTextLengthW(Dlg); Title = HeapAlloc(UserServerHeap, 0, (TitleLength + 1) * sizeof(WCHAR)); - if (NULL != Title) + if (Title) { Len = GetWindowTextW(Dlg, Title, TitleLength + 1); SendMessageW(NotifyContext->WndClient, WM_GETTEXT, - TitleLength + 1 - Len, (LPARAM) (Title + Len)); + TitleLength + 1 - Len, (LPARAM)(Title + Len)); SetWindowTextW(Dlg, Title); HeapFree(UserServerHeap, 0, Title); } @@ -137,7 +141,7 @@ break; case WM_TIMER: - NotifyContext = (PNOTIFY_CONTEXT) GetWindowLongPtrW(Dlg, DWLP_USER); + NotifyContext = (PNOTIFY_CONTEXT)GetWindowLongPtrW(Dlg, DWLP_USER); ProgressBar = GetDlgItem(Dlg, IDC_PROGRESS); UpdateProgressBar(ProgressBar, NotifyContext); Result = TRUE; @@ -146,8 +150,9 @@ case WM_COMMAND: if (BN_CLICKED == HIWORD(wParam) && IDC_END_NOW == LOWORD(wParam)) { - NotifyContext = (PNOTIFY_CONTEXT) GetWindowLongPtrW(Dlg, DWLP_USER); + NotifyContext = (PNOTIFY_CONTEXT)GetWindowLongPtrW(Dlg, DWLP_USER); NotifyContext->EndNowResult = QUERY_RESULT_FORCE; + MY_DPRINT("Closing progress dlg by hand\n"); SendMessageW(Dlg, WM_CLOSE, 0, 0); Result = TRUE; } @@ -158,12 +163,14 @@ break; case WM_CLOSE: + MY_DPRINT("WM_CLOSE\n"); DestroyWindow(Dlg); Result = TRUE; break; case WM_DESTROY: - NotifyContext = (PNOTIFY_CONTEXT) GetWindowLongPtrW(Dlg, DWLP_USER); + MY_DPRINT("WM_DESTROY\n"); + NotifyContext = (PNOTIFY_CONTEXT)GetWindowLongPtrW(Dlg, DWLP_USER); NotifyContext->Dlg = NULL; KillTimer(Dlg, 0); PostQuitMessage(NotifyContext->EndNowResult); @@ -178,48 +185,26 @@ return Result; } -HMODULE hComCtl32Lib = NULL; - -static VOID -CallInitCommonControls(VOID) -{ - static BOOL Initialized = FALSE; - INITCOMMONCONTROLS_PROC InitProc; - - if (Initialized) return; - - hComCtl32Lib = LoadLibraryW(L"COMCTL32.DLL"); - if (hComCtl32Lib == NULL) return; - - InitProc = (INITCOMMONCONTROLS_PROC)GetProcAddress(hComCtl32Lib, "InitCommonControls"); - if (InitProc == NULL) return; - - (*InitProc)(); - - Initialized = TRUE; -} - static DWORD WINAPI EndNowThreadProc(LPVOID Parameter) { - PNOTIFY_CONTEXT NotifyContext = (PNOTIFY_CONTEXT) Parameter; + PNOTIFY_CONTEXT NotifyContext = (PNOTIFY_CONTEXT)Parameter; MSG Msg; - SetThreadDesktop(NotifyContext->Desktop); - SwitchDesktop(NotifyContext->Desktop); + // SetThreadDesktop(NotifyContext->Desktop); + // SwitchDesktop(NotifyContext->Desktop); CallInitCommonControls(); NotifyContext->Dlg = CreateDialogParam(UserServerDllInstance, MAKEINTRESOURCE(IDD_END_NOW), NULL, - EndNowDlgProc, (LPARAM) NotifyContext); - if (NULL == NotifyContext->Dlg) - { + EndNowDlgProc, (LPARAM)NotifyContext); + if (NotifyContext->Dlg == NULL) return 0; - } + ShowWindow(NotifyContext->Dlg, SW_SHOWNORMAL); while (GetMessageW(&Msg, NULL, 0, 0)) { - if (! IsDialogMessage(NotifyContext->Dlg, &Msg)) + if (!IsDialogMessage(NotifyContext->Dlg, &Msg)) { TranslateMessage(&Msg); DispatchMessageW(&Msg); @@ -230,70 +215,73 @@ } static DWORD WINAPI -SendQueryEndSession(LPVOID Parameter) -{ - PMESSAGE_CONTEXT Context = (PMESSAGE_CONTEXT) Parameter; +SendClientShutdown(LPVOID Parameter) +{ + PMESSAGE_CONTEXT Context = (PMESSAGE_CONTEXT)Parameter; DWORD_PTR Result; - if (SendMessageTimeoutW(Context->Wnd, WM_QUERYENDSESSION, Context->wParam, - Context->lParam, SMTO_NORMAL, Context->Timeout, - &Result)) - { - return Result ? QUERY_RESULT_CONTINUE : QUERY_RESULT_ABORT; - } - - return 0 == GetLastError() ? QUERY_RESULT_TIMEOUT : QUERY_RESULT_ERROR; -} - -static DWORD WINAPI -SendEndSession(LPVOID Parameter) -{ - PMESSAGE_CONTEXT Context = (PMESSAGE_CONTEXT) Parameter; - DWORD_PTR Result; - - if (Context->wParam) - { - if (SendMessageTimeoutW(Context->Wnd, WM_ENDSESSION, Context->wParam, - Context->lParam, SMTO_NORMAL, Context->Timeout, - &Result)) - { - return QUERY_RESULT_CONTINUE; - } - return 0 == GetLastError() ? QUERY_RESULT_TIMEOUT : QUERY_RESULT_ERROR; - } - else - { - SendMessage(Context->Wnd, WM_ENDSESSION, Context->wParam, - Context->lParam); + /* If the shutdown is aborted, just notify the process, there is no need to wait */ + if ((Context->wParam & (MCS_QUERYENDSESSION | MCS_ENDSESSION)) == 0) + { + MY_DPRINT("Called WM_CLIENTSHUTDOWN with wParam == 0 ...\n"); + SendNotifyMessageW(Context->Wnd, WM_CLIENTSHUTDOWN, + Context->wParam, Context->lParam); return QUERY_RESULT_CONTINUE; } -} - -static BOOL CALLBACK -NotifyTopLevelEnum(HWND Wnd, LPARAM lParam) -{ - PNOTIFY_CONTEXT NotifyContext = (PNOTIFY_CONTEXT) lParam; + + if (SendMessageTimeoutW(Context->Wnd, WM_CLIENTSHUTDOWN, + Context->wParam, Context->lParam, + SMTO_NORMAL, Context->Timeout, &Result)) + { + DWORD Ret; + + if (Context->wParam & MCS_QUERYENDSESSION) + { + /* WM_QUERYENDSESSION case */ + switch (Result) + { + case MCSR_DONOTSHUTDOWN: + Ret = QUERY_RESULT_ABORT; + break; + + case MCSR_GOODFORSHUTDOWN: + case MCSR_SHUTDOWNFINISHED: + default: + Ret = QUERY_RESULT_CONTINUE; + } + } + else + { + /* WM_ENDSESSION case */ + Ret = QUERY_RESULT_CONTINUE; + } + + MY_DPRINT("SendClientShutdown -- Return == %s\n", + Ret == QUERY_RESULT_CONTINUE ? "Continue" : "Abort"); + return Ret; + } + + MY_DPRINT("SendClientShutdown -- Error == %s\n", + GetLastError() == 0 ? "Timeout" : "error"); + + return (GetLastError() == 0 ? QUERY_RESULT_TIMEOUT : QUERY_RESULT_ERROR); +} + +static BOOL +NotifyTopLevelWindow(HWND Wnd, PNOTIFY_CONTEXT NotifyContext) +{ MESSAGE_CONTEXT MessageContext; DWORD Now, Passed; DWORD Timeout, WaitStatus; - DWORD ProcessId; HANDLE MessageThread; HANDLE Threads[2]; - if (0 == GetWindowThreadProcessId(Wnd, &ProcessId)) - { - NotifyContext->QueryResult = QUERY_RESULT_ERROR; - return FALSE; - } - - if (ProcessId != NotifyContext->ProcessId) - goto Quit; + SetForegroundWindow(Wnd); Now = GetTickCount(); - if (0 == NotifyContext->StartTime) - { + if (NotifyContext->StartTime == 0) NotifyContext->StartTime = Now; - } + /* * Note: Passed is computed correctly even when GetTickCount() * wraps due to unsigned arithmetic. @@ -304,16 +292,16 @@ MessageContext.wParam = NotifyContext->wParam; MessageContext.lParam = NotifyContext->lParam; MessageContext.Timeout = NotifyContext->ShutdownSettings->HungAppTimeout; - if (! NotifyContext->ShutdownSettings->AutoEndTasks) + if (!NotifyContext->ShutdownSettings->AutoEndTasks) { MessageContext.Timeout += NotifyContext->ShutdownSettings->WaitToKillAppTimeout; } if (Passed < MessageContext.Timeout) { MessageContext.Timeout -= Passed; - MessageThread = CreateThread(NULL, 0, NotifyContext->SendMessageProc, - (LPVOID) &MessageContext, 0, NULL); - if (NULL == MessageThread) + MessageThread = CreateThread(NULL, 0, SendClientShutdown, + (LPVOID)&MessageContext, 0, NULL); + if (MessageThread == NULL) { NotifyContext->QueryResult = QUERY_RESULT_ERROR; return FALSE; @@ -331,30 +319,30 @@ if (WAIT_TIMEOUT == WaitStatus) { NotifyContext->WndClient = Wnd; - if (NULL == NotifyContext->UIThread && NotifyContext->ShowUI) + if (NotifyContext->UIThread == NULL && NotifyContext->ShowUI) { NotifyContext->UIThread = CreateThread(NULL, 0, EndNowThreadProc, - (LPVOID) NotifyContext, + (LPVOID)NotifyContext, 0, NULL); } Threads[0] = MessageThread; Threads[1] = NotifyContext->UIThread; - WaitStatus = WaitForMultipleObjectsEx(NULL == NotifyContext->UIThread ? + WaitStatus = WaitForMultipleObjectsEx(NotifyContext->UIThread == NULL ? 1 : 2, Threads, FALSE, INFINITE, FALSE); - if (WAIT_OBJECT_0 == WaitStatus) - { - if (! GetExitCodeThread(MessageThread, &NotifyContext->QueryResult)) + if (WaitStatus == WAIT_OBJECT_0) + { + if (!GetExitCodeThread(MessageThread, &NotifyContext->QueryResult)) { NotifyContext->QueryResult = QUERY_RESULT_ERROR; } } - else if (WAIT_OBJECT_0 + 1 == WaitStatus) - { - if (! GetExitCodeThread(NotifyContext->UIThread, - &NotifyContext->QueryResult)) + else if (WaitStatus == WAIT_OBJECT_0 + 1) + { + if (!GetExitCodeThread(NotifyContext->UIThread, + &NotifyContext->QueryResult)) { NotifyContext->QueryResult = QUERY_RESULT_ERROR; } @@ -363,15 +351,15 @@ { NotifyContext->QueryResult = QUERY_RESULT_ERROR; } - if (WAIT_OBJECT_0 != WaitStatus) + if (WaitStatus != WAIT_OBJECT_0) { TerminateThread(MessageThread, QUERY_RESULT_TIMEOUT); } } - else if (WAIT_OBJECT_0 == WaitStatus) - { - if (! GetExitCodeThread(MessageThread, - &NotifyContext->QueryResult)) + else if (WaitStatus == WAIT_OBJECT_0) + { + if (!GetExitCodeThread(MessageThread, + &NotifyContext->QueryResult)) { NotifyContext->QueryResult = QUERY_RESULT_ERROR; } @@ -387,59 +375,16 @@ NotifyContext->QueryResult = QUERY_RESULT_TIMEOUT; } -Quit: - DPRINT1("NotifyContext->QueryResult == %d\n", NotifyContext->QueryResult); + MY_DPRINT("NotifyContext->QueryResult == %d\n", NotifyContext->QueryResult); return (NotifyContext->QueryResult == QUERY_RESULT_CONTINUE); } -static BOOL CALLBACK -NotifyDesktopEnum(LPWSTR DesktopName, LPARAM lParam) -{ - PNOTIFY_CONTEXT Context = (PNOTIFY_CONTEXT) lParam; - - Context->Desktop = OpenDesktopW(DesktopName, 0, FALSE, - DESKTOP_ENUMERATE | DESKTOP_SWITCHDESKTOP); - if (NULL == Context->Desktop) - { - DPRINT1("OpenDesktop failed with error %d\n", GetLastError()); - Context->QueryResult = QUERY_RESULT_ERROR; - return FALSE; - } - - Context->OldDesktop = GetThreadDesktop(GetCurrentThreadId()); - SwitchDesktop(Context->Desktop); - - EnumDesktopWindows(Context->Desktop, NotifyTopLevelEnum, lParam); - - SwitchDesktop(Context->OldDesktop); - - CloseDesktop(Context->Desktop); - - return QUERY_RESULT_CONTINUE == Context->QueryResult; -} - -static BOOL FASTCALL -NotifyTopLevelWindows(PNOTIFY_CONTEXT Context) -{ - HWINSTA WindowStation; - - WindowStation = GetProcessWindowStation(); - if (NULL == WindowStation) - { - DPRINT1("GetProcessWindowStation failed with error %d\n", GetLastError()); - return TRUE; - } - - EnumDesktopsW(WindowStation, NotifyDesktopEnum, (LPARAM) Context); - - return TRUE; -} - -static BOOL -DtbgIsDesktopVisible(VOID) -{ - return !((BOOL)NtUserCallNoParam(NOPARAM_ROUTINE_ISCONSOLEMODE)); -} +static BOOLEAN +IsConsoleMode(VOID) +{ + return (BOOLEAN)NtUserCallNoParam(NOPARAM_ROUTINE_ISCONSOLEMODE); +} + /* TODO: Find an other way to do it. */ #if 0 @@ -452,11 +397,10 @@ if (ProcessData->CtrlDispatcher) { - Thread = CreateRemoteThread(ProcessData->ProcessHandle, NULL, 0, (LPTHREAD_START_ROUTINE) ProcessData->CtrlDispatcher, UlongToPtr(Event), 0, NULL); - if (NULL == Thread) + if (Thread == NULL) { DPRINT1("Failed thread creation (Error: 0x%x)\n", GetLastError()); return; @@ -468,296 +412,221 @@ #endif /************************************************/ -BOOL FASTCALL -NotifyAndTerminateProcess(PCSR_PROCESS ProcessData, - PSHUTDOWN_SETTINGS ShutdownSettings, - UINT Flags) -{ - NOTIFY_CONTEXT Context; - HANDLE Process; + +static BOOL CALLBACK +FindTopLevelWnd(IN HWND hWnd, + IN LPARAM lParam) +{ + if (GetWindow(hWnd, GW_OWNER) == NULL) + { + *(HWND*)lParam = hWnd; + return FALSE; + } + return TRUE; +} + +static VOID +ThreadShutdownNotify(IN PCSR_THREAD CsrThread, + IN ULONG Flags, + IN ULONG Flags2, + IN PNOTIFY_CONTEXT Context) +{ + HWND TopWnd = NULL; + + EnumThreadWindows(HandleToUlong(CsrThread->ClientId.UniqueThread), + FindTopLevelWnd, (LPARAM)&TopWnd); + if (TopWnd) + { + HWND hWndOwner; + + /*** FOR TESTING PURPOSES ONLY!! ***/ + HWND tmpWnd; + tmpWnd = TopWnd; + /***********************************/ + + while ((hWndOwner = GetWindow(TopWnd, GW_OWNER)) != NULL) + { + MY_DPRINT("GetWindow(TopWnd, GW_OWNER) not returned NULL...\n"); + TopWnd = hWndOwner; + } + if (TopWnd != tmpWnd) MY_DPRINT("(TopWnd = %x) != (tmpWnd = %x)\n", TopWnd, tmpWnd); + } + if (TopWnd == NULL) + return; + + Context->wParam = Flags2; + Context->lParam = (0 != (Flags & EWX_CALLER_WINLOGON_LOGOFF) ? + ENDSESSION_LOGOFF : 0); + + Context->StartTime = 0; + Context->UIThread = NULL; + Context->ShowUI = !IsConsoleMode() && (Flags2 & (MCS_QUERYENDSESSION | MCS_ENDSESSION)); + Context->Dlg = NULL; + +#if 0 // Obviously, switching desktops like that from within WINSRV doesn't work... + { + BOOL Success; + Context->OldDesktop = GetThreadDesktop(GetCurrentThreadId()); + // Context->Desktop = GetThreadDesktop(HandleToUlong(CsrThread->ClientId.UniqueThread)); + Context->Desktop = GetThreadDesktop(GetWindowThreadProcessId(TopWnd, NULL)); + MY_DPRINT("Last error = %d\n", GetLastError()); + MY_DPRINT("Before switching to desktop 0x%x\n", Context->Desktop); + Success = SwitchDesktop(Context->Desktop); + MY_DPRINT("After switching to desktop (Success = %s ; last error = %d); going to notify top-level...\n", + Success ? "TRUE" : "FALSE", GetLastError()); + } +#endif + + NotifyTopLevelWindow(TopWnd, Context); + +/******************************************************************************/ +#if 1 + if (Context->UIThread) + { + MY_DPRINT("Context->UIThread != NULL\n"); + if (Context->Dlg) + { + MY_DPRINT("Sending WM_CLOSE because Dlg is != NULL\n"); + SendMessageW(Context->Dlg, WM_CLOSE, 0, 0); + } + else + { + MY_DPRINT("Terminating UIThread thread with QUERY_RESULT_ERROR\n"); + TerminateThread(Context->UIThread, QUERY_RESULT_ERROR); + } + CloseHandle(Context->UIThread); + /**/Context->UIThread = NULL;/**/ + /**/Context->Dlg = NULL;/**/ + } +#endif +/******************************************************************************/ + +#if 0 + MY_DPRINT("Switch back to old desktop 0x%x\n", Context->OldDesktop); + SwitchDesktop(Context->OldDesktop); + MY_DPRINT("Switched back ok\n"); +#endif +} + +static BOOL +NotifyProcessForShutdown(PCSR_PROCESS CsrProcess, + PSHUTDOWN_SETTINGS ShutdownSettings, + UINT Flags) +{ DWORD QueryResult = QUERY_RESULT_CONTINUE; - Context.QueryResult = QUERY_RESULT_CONTINUE; - - if (0 == (Flags & EWX_FORCE)) - { - // TODO: Find an other way whether or not the process has a console. + /* In case we make a forced shutdown, just kill the process */ + if (Flags & EWX_FORCE) + return TRUE; + + // TODO: Find an other way whether or not the process has a console. #if 0 - if (NULL != ProcessData->Console) - { - ConioConsoleCtrlEventTimeout(CTRL_LOGOFF_EVENT, ProcessData, - ShutdownSettings->WaitToKillAppTimeout); - } - else + if (CsrProcess->Console) + { + ConioConsoleCtrlEventTimeout(CTRL_LOGOFF_EVENT, CsrProcess, + ShutdownSettings->WaitToKillAppTimeout); + } + else #endif - { - Context.ProcessId = (DWORD_PTR) ProcessData->ClientId.UniqueProcess; - Context.wParam = 0; - Context.lParam = (0 != (Flags & EWX_CALLER_WINLOGON_LOGOFF) ? - ENDSESSION_LOGOFF : 0); - Context.StartTime = 0; - Context.UIThread = NULL; - Context.ShowUI = DtbgIsDesktopVisible(); - Context.Dlg = NULL; - Context.ShutdownSettings = ShutdownSettings; - Context.SendMessageProc = SendQueryEndSession; - - NotifyTopLevelWindows(&Context); - - Context.wParam = (QUERY_RESULT_ABORT != Context.QueryResult); - Context.lParam = (0 != (Flags & EWX_CALLER_WINLOGON_LOGOFF) ? - ENDSESSION_LOGOFF : 0); - Context.SendMessageProc = SendEndSession; - Context.ShowUI = DtbgIsDesktopVisible() && - (QUERY_RESULT_ABORT != Context.QueryResult); - QueryResult = Context.QueryResult; + { + PCSR_PROCESS Process; + PCSR_THREAD Thread; + PLIST_ENTRY NextEntry; + + NOTIFY_CONTEXT Context; + Context.ShutdownSettings = ShutdownSettings; + + /* Lock the process */ + CsrLockProcessByClientId(CsrProcess->ClientId.UniqueProcess, &Process); + + /* Send first the QUERYENDSESSION messages to all the threads of the process */ + MY_DPRINT2("Sending the QUERYENDSESSION messages...\n"); + + NextEntry = CsrProcess->ThreadList.Flink; + while (NextEntry != &CsrProcess->ThreadList) + { + /* Get the current thread entry */ + Thread = CONTAINING_RECORD(NextEntry, CSR_THREAD, Link); + + /* Move to the next entry */ + NextEntry = NextEntry->Flink; + + /* If the thread is being terminated, just skip it */ + if (Thread->Flags & CsrThreadTerminated) continue; + + /* Reference the thread and temporarily unlock the process */ + CsrReferenceThread(Thread); + CsrUnlockProcess(Process); + Context.QueryResult = QUERY_RESULT_CONTINUE; - - NotifyTopLevelWindows(&Context); - - if (NULL != Context.UIThread) - { - if (NULL != Context.Dlg) - { - SendMessageW(Context.Dlg, WM_CLOSE, 0, 0); - } - else - { - TerminateThread(Context.UIThread, QUERY_RESULT_ERROR); - } - CloseHandle(Context.UIThread); - } - } - - if (QUERY_RESULT_ABORT == QueryResult) - { - return FALSE; - } - } - - /* Terminate this process */ - Process = OpenProcess(PROCESS_TERMINATE, FALSE, - (DWORD_PTR) ProcessData->ClientId.UniqueProcess); - if (NULL == Process) - { - DPRINT1("Unable to open process %d, error %d\n", ProcessData->ClientId.UniqueProcess, - GetLastError()); - return TRUE; - } - TerminateProcess(Process, 0); - CloseHandle(Process); - - return TRUE; -} + ThreadShutdownNotify(Thread, Flags, + MCS_QUERYENDSESSION, + &Context); + + /* Lock the process again and dereference the thread */ + CsrLockProcessByClientId(CsrProcess->ClientId.UniqueProcess, &Process); + CsrDereferenceThread(Thread); + + // FIXME: Analyze Context.QueryResult !! + /**/if (Context.QueryResult == QUERY_RESULT_ABORT) goto Quit;/**/ + } + + QueryResult = Context.QueryResult; + MY_DPRINT2("QueryResult = %s\n", + QueryResult == QUERY_RESULT_ABORT ? "Abort" : "Continue"); + + /* Now send the ENDSESSION messages to the threads */ + MY_DPRINT2("Now sending the ENDSESSION messages...\n"); + + NextEntry = CsrProcess->ThreadList.Flink; + while (NextEntry != &CsrProcess->ThreadList) + { + /* Get the current thread entry */ + Thread = CONTAINING_RECORD(NextEntry, CSR_THREAD, Link); + + /* Move to the next entry */ + NextEntry = NextEntry->Flink; + + /* If the thread is being terminated, just skip it */ + if (Thread->Flags & CsrThreadTerminated) continue; + + /* Reference the thread and temporarily unlock the process */ + CsrReferenceThread(Thread); + CsrUnlockProcess(Process); + + Context.QueryResult = QUERY_RESULT_CONTINUE; + ThreadShutdownNotify(Thread, Flags, + (QUERY_RESULT_ABORT != QueryResult) ? MCS_ENDSESSION : 0, + &Context); + + /* Lock the process again and dereference the thread */ + CsrLockProcessByClientId(CsrProcess->ClientId.UniqueProcess, &Process); + CsrDereferenceThread(Thread); + } + +Quit: + /* Unlock the process */ + CsrUnlockProcess(Process); #if 0 -static NTSTATUS WINAPI -ExitReactosProcessEnum(PCSR_PROCESS ProcessData, PVOID Data) -{ - HANDLE Process; - HANDLE Token; - TOKEN_ORIGIN Origin; - DWORD ReturnLength; - PPROCESS_ENUM_CONTEXT Context = (PPROCESS_ENUM_CONTEXT) Data; - PCSR_PROCESS *NewData; - - /* Do not kill winlogon or csrss */ - if ((DWORD_PTR) ProcessData->ClientId.UniqueProcess == Context->CsrssProcess || - ProcessData->ClientId.UniqueProcess == LogonProcessId) - { - return STATUS_SUCCESS; - } - - /* Get the login session of this process */ - Process = OpenProcess(PROCESS_QUERY_INFORMATION, FALSE, - (DWORD_PTR) ProcessData->ClientId.UniqueProcess); - if (NULL == Process) - { - DPRINT1("Unable to open process %d, error %d\n", ProcessData->ClientId.UniqueProcess, - GetLastError()); - return STATUS_UNSUCCESSFUL; - } - - if (! OpenProcessToken(Process, TOKEN_QUERY, &Token)) - { - DPRINT1("Unable to open token for process %d, error %d\n", - ProcessData->ClientId.UniqueProcess, GetLastError()); - CloseHandle(Process); - return STATUS_UNSUCCESSFUL; - } - CloseHandle(Process); - - if (! GetTokenInformation(Token, TokenOrigin, &Origin, - sizeof(TOKEN_ORIGIN), &ReturnLength)) - { - DPRINT1("GetTokenInformation failed for process %d with error %d\n", - ProcessData->ClientId.UniqueProcess, GetLastError()); - CloseHandle(Token); - return STATUS_UNSUCCESSFUL; - } - CloseHandle(Token); - - /* This process will be killed if it's in the correct logon session */ - if (RtlEqualLuid(&(Context->TokenOrigin.OriginatingLogonSession), - &(Origin.OriginatingLogonSession))) - { - /* Kill the shell process last */ - if ((DWORD_PTR) ProcessData->ClientId.UniqueProcess == Context->ShellProcess) - { - ProcessData->ShutdownLevel = 0; - } - NewData = HeapAlloc(UserServerHeap, 0, (Context->ProcessCount + 1) - * sizeof(PCSR_PROCESS)); - if (NULL == NewData) - { - return STATUS_NO_MEMORY; - } - if (0 != Context->ProcessCount) - { - memcpy(NewData, Context->ProcessData, - Context->ProcessCount * sizeof(PCSR_PROCESS)); - HeapFree(UserServerHeap, 0, Context->ProcessData); - } - Context->ProcessData = NewData; - Context->ProcessData[Context->ProcessCount] = ProcessData; - Context->ProcessCount++; - } - - return STATUS_SUCCESS; -} + if (Context.UIThread) + { + if (Context.Dlg) + { + SendMessageW(Context.Dlg, WM_CLOSE, 0, 0); + } + else + { + TerminateThread(Context.UIThread, QUERY_RESULT_ERROR); + } + CloseHandle(Context.UIThread); + } #endif - -#if 0 -static DWORD FASTCALL -GetShutdownSettings(HKEY DesktopKey, LPCWSTR ValueName, DWORD DefaultValue) -{ - BYTE ValueBuffer[16]; - LONG ErrCode; - DWORD Type; - DWORD ValueSize; - UNICODE_STRING StringValue; - ULONG Value; - - ValueSize = sizeof(ValueBuffer); - ErrCode = RegQueryValueExW(DesktopKey, ValueName, NULL, &Type, ValueBuffer, - &ValueSize); - if (ERROR_SUCCESS != ErrCode) - { - DPRINT("GetShutdownSettings for %S failed with error code %ld\n", - ValueName, ErrCode); - return DefaultValue; - } - - if (REG_SZ == Type) - { - RtlInitUnicodeString(&StringValue, (LPCWSTR) ValueBuffer); - if (! NT_SUCCESS(RtlUnicodeStringToInteger(&StringValue, 10, &Value))) - { - DPRINT1("Unable to convert value %S for setting %S\n", - StringValue.Buffer, ValueName); - return DefaultValue; - } - return (DWORD) Value; - } - else if (REG_DWORD == Type) - { - return *((DWORD *) ValueBuffer); - } - - DPRINT1("Unexpected registry type %d for setting %S\n", Type, ValueName); - return DefaultValue; -} - -static VOID FASTCALL -LoadShutdownSettings(PSID Sid, PSHUTDOWN_SETTINGS ShutdownSettings) -{ - static WCHAR Subkey[] = L"\\Control Panel\\Desktop"; - LPWSTR StringSid; - WCHAR InitialKeyName[128]; - LPWSTR KeyName; - HKEY DesktopKey; - LONG ErrCode; - - ShutdownSettings->AutoEndTasks = DEFAULT_AUTO_END_TASKS; - ShutdownSettings->HungAppTimeout = DEFAULT_HUNG_APP_TIMEOUT; - ShutdownSettings->WaitToKillAppTimeout = DEFAULT_WAIT_TO_KILL_APP_TIMEOUT; - - if (! ConvertSidToStringSidW(Sid, &StringSid)) - { - DPRINT1("ConvertSidToStringSid failed with error %d, using default shutdown settings\n", - GetLastError()); - return; - } - if (wcslen(StringSid) + wcslen(Subkey) + 1 <= - sizeof(InitialKeyName) / sizeof(WCHAR)) - { - KeyName = InitialKeyName; - } - else - { - KeyName = HeapAlloc(UserServerHeap, 0, - (wcslen(StringSid) + wcslen(Subkey) + 1) * - sizeof(WCHAR)); - if (NULL == KeyName) - { - DPRINT1("Failed to allocate memory, using default shutdown settings\n"); - LocalFree(StringSid); - return; - } - } - wcscat(wcscpy(KeyName, StringSid), Subkey); - LocalFree(StringSid); - - ErrCode = RegOpenKeyExW(HKEY_USERS, KeyName, 0, KEY_QUERY_VALUE, &DesktopKey); - if (KeyName != InitialKeyName) - { - HeapFree(UserServerHeap, 0, KeyName); - } - if (ERROR_SUCCESS != ErrCode) - { - DPRINT1("RegOpenKeyEx failed with error %ld, using default shutdown settings\n", ErrCode); - return; - } - - ShutdownSettings->AutoEndTasks = (BOOL) GetShutdownSettings(DesktopKey, L"AutoEndTasks", - (DWORD) DEFAULT_AUTO_END_TASKS); - ShutdownSettings->HungAppTimeout = GetShutdownSettings(DesktopKey, - L"HungAppTimeout", - DEFAULT_HUNG_APP_TIMEOUT); - ShutdownSettings->WaitToKillAppTimeout = GetShutdownSettings(DesktopKey, - L"WaitToKillAppTimeout", - DEFAULT_WAIT_TO_KILL_APP_TIMEOUT); - - RegCloseKey(DesktopKey); -} - -NTSTATUS FASTCALL -InternalExitReactos(DWORD ProcessId, DWORD ThreadId, UINT Flags) -{ - // PROCESS_ENUM_CONTEXT Context; - HWND ShellWnd; - char FixedUserInfo[64]; - TOKEN_USER *UserInfo; - SHUTDOWN_SETTINGS ShutdownSettings; - - LoadShutdownSettings(UserInfo->User.Sid, &ShutdownSettings); - - // Context.CsrssProcess = GetCurrentProcessId(); - // ShellWnd = GetShellWindow(); - // if (NULL == ShellWnd) - // { - // DPRINT("No shell present\n"); - // Context.ShellProcess = 0; - // } - // else if (0 == GetWindowThreadProcessId(ShellWnd, &Context.ShellProcess)) - // { - // DPRINT1("Can't get process id of shell window\n"); - // Context.ShellProcess = 0; - // } - - return STATUS_SUCCESS; -} -#endif + } + + /* Kill the process unless we abort shutdown */ + return (QueryResult != QUERY_RESULT_ABORT); +} static NTSTATUS FASTCALL UserExitReactos(PCSR_THREAD CsrThread, UINT Flags) @@ -893,7 +762,7 @@ * Check for process validity */ - /* Do not kill WinLogon or CSRSS */ + /* Do not kill Winlogon or CSRSS */ if (CsrProcess->ClientId.UniqueProcess == NtCurrentProcess() || CsrProcess->ClientId.UniqueProcess == UlongToHandle(LogonProcessId)) { @@ -909,14 +778,25 @@ else DPRINT1("Killing process with ShutdownFlags = %lu\n", CsrProcess->ShutdownFlags); -#if 0 - if (!NotifyAndTerminateProcess(CsrProcess, &ShutdownSettings, Flags)) - { + if (CsrProcess->ShutdownFlags & SHUTDOWN_OTHERCONTEXT) + DPRINT1("This process has SHUTDOWN_OTHERCONTEXT\n"); + + if (CsrProcess->ShutdownFlags & SHUTDOWN_SYSTEMCONTEXT) + DPRINT1("This process has SHUTDOWN_SYSTEMCONTEXT\n"); + + /* Notify the process for shutdown if needed */ + if (!NotifyProcessForShutdown(CsrProcess, &ShutdownSettings, Flags)) + { + /* Abort shutdown */ return CsrShutdownCancelled; } -#endif - - return CsrShutdownNonCsrProcess; + + /* Terminate this process */ + NtTerminateProcess(CsrProcess->ProcessHandle, 0); + + /* We are done */ + CsrDereferenceProcess(CsrProcess); + return CsrShutdownCsrProcess; } @@ -965,16 +845,10 @@ return STATUS_SUCCESS; } -CSR_API(SrvLogon) +CSR_API(SrvRecordShutdownReason) { DPRINT1("%s not yet implemented\n", __FUNCTION__); return STATUS_NOT_IMPLEMENTED; } -CSR_API(SrvRecordShutdownReason) -{ - DPRINT1("%s not yet implemented\n", __FUNCTION__); - return STATUS_NOT_IMPLEMENTED; -} - /* EOF */ Modified: trunk/reactos/win32ss/user/winsrv/winsrv.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/winsrv/winsrv… ============================================================================== --- trunk/reactos/win32ss/user/winsrv/winsrv.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/winsrv/winsrv.h [iso-8859-1] Sun Feb 15 22:57:40 2015 @@ -20,6 +20,7 @@ #include <windef.h> #include <winbase.h> #include <wingdi.h> +#include <winreg.h> #include <winuser.h> #include <imm.h> @@ -42,4 +43,18 @@ /* CSRSS Header */ #include <csr/csrsrv.h> +typedef struct tagSHUTDOWN_SETTINGS +{ + BOOL AutoEndTasks; + ULONG HungAppTimeout; + ULONG WaitToKillAppTimeout; + ULONG WaitToKillServiceTimeout; + ULONG ProcessTerminateTimeout; +} SHUTDOWN_SETTINGS, *PSHUTDOWN_SETTINGS; + +extern SHUTDOWN_SETTINGS ShutdownSettings; + +VOID FASTCALL +GetTimeouts(IN PSHUTDOWN_SETTINGS ShutdownSettings); + #endif /* __WINSRV_H__ */

1 0

[hbelusca] 66305: [NTOS]: Less "magic" indices.
by hbelusca＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: hbelusca Date: Sun Feb 15 22:29:50 2015 New Revision: 66305 URL: http://svn.reactos.org/svn/reactos?rev=66305&view=rev Log: [NTOS]: Less "magic" indices. Modified: trunk/reactos/ntoskrnl/ke/thrdobj.c Modified: trunk/reactos/ntoskrnl/ke/thrdobj.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/thrdobj.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/ke/thrdobj.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ke/thrdobj.c [iso-8859-1] Sun Feb 15 22:29:50 2015 @@ -803,8 +803,8 @@ Thread->ServiceTable = KeServiceDescriptorTable; /* Setup APC Fields */ - InitializeListHead(&Thread->ApcState.ApcListHead[0]); - InitializeListHead(&Thread->ApcState.ApcListHead[1]); + InitializeListHead(&Thread->ApcState.ApcListHead[KernelMode]); + InitializeListHead(&Thread->ApcState.ApcListHead[UserMode]); Thread->ApcState.Process = Process; Thread->ApcStatePointer[OriginalApcEnvironment] = &Thread->ApcState; Thread->ApcStatePointer[AttachedApcEnvironment] = &Thread->SavedApcState;

1 0

[tfaber] 66304: [NTOS] - In ExpReleaseOrWaitForKeyedEvent, always restore previous values for ETHREAD::KeyedWaitValue and ETHREAD::KeyedWaitChain, as these fields have alternative meanings outside ...
by tfaber＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: tfaber Date: Sun Feb 15 22:20:34 2015 New Revision: 66304 URL: http://svn.reactos.org/svn/reactos?rev=66304&view=rev Log: [NTOS] - In ExpReleaseOrWaitForKeyedEvent, always restore previous values for ETHREAD::KeyedWaitValue and ETHREAD::KeyedWaitChain, as these fields have alternative meanings outside of keyed events - Add missing list walk in ExpReleaseOrWaitForKeyedEvent - In Nt(WaitFor|Release)KeyedEvent, refuse keys that are not two-byte aligned as shown by Wine tests - Don't forget to set KTHREAD::Process on thread creation Fixes hang when running ntdll_winetest:om as well as failing tests. ROSTESTS-118 #resolve Modified: trunk/reactos/ntoskrnl/ex/keyedevt.c trunk/reactos/ntoskrnl/ke/thrdobj.c trunk/rostests/winetests/ntdll/om.c Modified: trunk/reactos/ntoskrnl/ex/keyedevt.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ex/keyedevt.c?rev… ============================================================================== --- trunk/reactos/ntoskrnl/ex/keyedevt.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ex/keyedevt.c [iso-8859-1] Sun Feb 15 22:20:34 2015 @@ -132,6 +132,7 @@ PLIST_ENTRY ListEntry, WaitListHead1, WaitListHead2; NTSTATUS Status; ULONG_PTR HashIndex; + PVOID PreviousKeyedWaitValue; /* Get the current process */ CurrentProcess = KeGetCurrentProcess(); @@ -167,6 +168,7 @@ be signaled by this thread or, when the wait is aborted due to thread termination, then it first needs to acquire the list lock. */ Thread = CONTAINING_RECORD(ListEntry, ETHREAD, KeyedWaitChain); + ListEntry = ListEntry->Flink; /* Check if this thread is a correct waiter */ if ((Thread->Tcb.Process == CurrentProcess) && @@ -179,7 +181,10 @@ InitializeListHead(&Thread->KeyedWaitChain); /* Wake the thread */ - KeReleaseSemaphore(&Thread->KeyedWaitSemaphore, 0, 1, FALSE); + KeReleaseSemaphore(&Thread->KeyedWaitSemaphore, + IO_NO_INCREMENT, + 1, + FALSE); Thread = NULL; /* Unlock the list. After this it is not safe to access Thread */ @@ -193,7 +198,8 @@ /* Get the current thread */ CurrentThread = PsGetCurrentThread(); - /* Set the wait key */ + /* Set the wait key and remember the old value */ + PreviousKeyedWaitValue = CurrentThread->KeyedWaitValue; CurrentThread->KeyedWaitValue = KeyedWaitValue; /* Initialize the wait semaphore */ @@ -221,16 +227,20 @@ ExAcquirePushLockExclusive(&KeyedEvent->HashTable[HashIndex].Lock); /* Check if the wait list entry is still in the list */ - if (CurrentThread->KeyedWaitChain.Flink != &CurrentThread->KeyedWaitChain) + if (!IsListEmpty(&CurrentThread->KeyedWaitChain)) { /* Remove the thread from the list */ RemoveEntryList(&CurrentThread->KeyedWaitChain); + InitializeListHead(&CurrentThread->KeyedWaitChain); } /* Unlock the list */ ExReleasePushLockExclusive(&KeyedEvent->HashTable[HashIndex].Lock); KeLeaveCriticalRegion(); } + + /* Restore the previous KeyedWaitValue, since this is a union member */ + CurrentThread->KeyedWaitValue = PreviousKeyedWaitValue; return Status; } @@ -412,6 +422,12 @@ NTSTATUS Status; LARGE_INTEGER TimeoutCopy; + /* Key must always be two-byte aligned */ + if ((ULONG_PTR)Key & 1) + { + return STATUS_INVALID_PARAMETER_1; + } + /* Check if the caller passed a timeout value and this is from user mode */ if ((Timeout != NULL) && (PreviousMode != KernelMode)) { @@ -472,6 +488,12 @@ NTSTATUS Status; LARGE_INTEGER TimeoutCopy; + /* Key must always be two-byte aligned */ + if ((ULONG_PTR)Key & 1) + { + return STATUS_INVALID_PARAMETER_1; + } + /* Check if the caller passed a timeout value and this is from user mode */ if ((Timeout != NULL) && (PreviousMode != KernelMode)) { Modified: trunk/reactos/ntoskrnl/ke/thrdobj.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/ntoskrnl/ke/thrdobj.c?rev=… ============================================================================== --- trunk/reactos/ntoskrnl/ke/thrdobj.c [iso-8859-1] (original) +++ trunk/reactos/ntoskrnl/ke/thrdobj.c [iso-8859-1] Sun Feb 15 22:20:34 2015 @@ -838,8 +838,9 @@ TimerWaitBlock->WaitListEntry.Flink = &Timer->Header.WaitListHead; TimerWaitBlock->WaitListEntry.Blink = &Timer->Header.WaitListHead; - /* Set the TEB */ + /* Set the TEB and process */ Thread->Teb = Teb; + Thread->Process = Process; /* Check if we have a kernel stack */ if (!KernelStack) Modified: trunk/rostests/winetests/ntdll/om.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/ntdll/om.c?rev=… ============================================================================== --- trunk/rostests/winetests/ntdll/om.c [iso-8859-1] (original) +++ trunk/rostests/winetests/ntdll/om.c [iso-8859-1] Sun Feb 15 22:20:34 2015 @@ -1080,8 +1080,5 @@ test_query_object(); test_type_mismatch(); test_event(); - if (winetest_interactive) - test_keyed_events(); - else - skip("Skipping test_keyed_events(). ROSTESTS-118.\n"); -} + test_keyed_events(); +}

1 0

[hbelusca] 66303: [USER32] Implement SrvLogon (client-side), called by the SetWindowStationUser API (called by winlogon) when a user logs-on (or off). See also http://www.openrce.org/reference_libr...
by hbelusca＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: hbelusca Date: Sun Feb 15 21:56:03 2015 New Revision: 66303 URL: http://svn.reactos.org/svn/reactos?rev=66303&view=rev Log: [USER32] Implement SrvLogon (client-side), called by the SetWindowStationUser API (called by winlogon) when a user logs-on (or off). See also http://www.openrce.org/reference_library/win32_call_chains/2003SP1/USER32/S… and http://www.openrce.org/reference_library/win32_call_chains/2003SP1/USER32/L… for details. Part 1/2 Modified: trunk/reactos/include/reactos/subsys/win/winmsg.h trunk/reactos/win32ss/user/user32/include/user32p.h trunk/reactos/win32ss/user/user32/misc/logon.c trunk/reactos/win32ss/user/user32/misc/winsta.c Modified: trunk/reactos/include/reactos/subsys/win/winmsg.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/include/reactos/subsys/win… ============================================================================== --- trunk/reactos/include/reactos/subsys/win/winmsg.h [iso-8859-1] (original) +++ trunk/reactos/include/reactos/subsys/win/winmsg.h [iso-8859-1] Sun Feb 15 21:56:03 2015 @@ -60,6 +60,11 @@ BOOL Success; } USER_END_TASK, *PUSER_END_TASK; +typedef struct _USER_LOGON +{ + BOOL IsLogon; +} USER_LOGON, *PUSER_LOGON; + typedef struct _USER_GET_THREAD_CONSOLE_DESKTOP { ULONG_PTR ThreadId; @@ -90,6 +95,7 @@ { USER_EXIT_REACTOS ExitReactosRequest; USER_END_TASK EndTaskRequest; + USER_LOGON LogonRequest; USER_GET_THREAD_CONSOLE_DESKTOP GetThreadConsoleDesktopRequest; USER_REGISTER_SERVICES_PROCESS RegisterServicesProcessRequest; USER_REGISTER_LOGON_PROCESS RegisterLogonProcessRequest; Modified: trunk/reactos/win32ss/user/user32/include/user32p.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/includ… ============================================================================== --- trunk/reactos/win32ss/user/user32/include/user32p.h [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/user32/include/user32p.h [iso-8859-1] Sun Feb 15 21:56:03 2015 @@ -113,6 +113,9 @@ VOID MenuTrackMouseMenuBar(HWND hWnd, ULONG Ht, POINT Pt); VOID MenuTrackKbdMenuBar(HWND hWnd, UINT wParam, WCHAR wChar); +/* definitions for logon.c */ +VOID FASTCALL Logon(BOOL IsLogon); + /* misc definitions */ void mirror_rect( const RECT *window_rect, RECT *rect ); BOOL FASTCALL DefSetText(HWND hWnd, PCWSTR String, BOOL Ansi); Modified: trunk/reactos/win32ss/user/user32/misc/logon.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/misc/l… ============================================================================== --- trunk/reactos/win32ss/user/user32/misc/logon.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/user32/misc/logon.c [iso-8859-1] Sun Feb 15 21:56:03 2015 @@ -74,6 +74,22 @@ } /* + * Helper function used by SetWindowStationUser (see winsta.c) + */ +VOID FASTCALL +Logon(BOOL IsLogon) +{ + USER_API_MESSAGE ApiMessage; + PUSER_LOGON LogonRequest = &ApiMessage.Data.LogonRequest; + + LogonRequest->IsLogon = IsLogon; + CsrClientCallServer((PCSR_API_MESSAGE)&ApiMessage, + NULL, + CSR_CREATE_API_NUMBER(USERSRV_SERVERDLL_INDEX, UserpLogon), + sizeof(*LogonRequest)); +} + +/* * @implemented */ BOOL Modified: trunk/reactos/win32ss/user/user32/misc/winsta.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/win32ss/user/user32/misc/w… ============================================================================== --- trunk/reactos/win32ss/user/user32/misc/winsta.c [iso-8859-1] (original) +++ trunk/reactos/win32ss/user/user32/misc/winsta.c [iso-8859-1] Sun Feb 15 21:56:03 2015 @@ -371,7 +371,7 @@ /* - * @unimplemented + * @implemented */ BOOL WINAPI @@ -380,7 +380,21 @@ PSID psid, DWORD size) { - return NtUserSetWindowStationUser(hWindowStation, pluid, psid, size); + BOOL Success; + + Success = NtUserSetWindowStationUser(hWindowStation, pluid, psid, size); + if (Success) + { + /* Signal log-on/off to WINSRV */ + + /* User is logging on if pluid != LuidNone, otherwise it is a log-off */ + LUID LuidNone = {0, 0}; + BOOL IsLogon = (pluid && !RtlEqualLuid(pluid, &LuidNone)); + + Logon(IsLogon); + } + + return Success; } /* EOF */

1 0

[tfaber] 66302: [ADVAPI32_WINETEST] - Skip over service timeout tests because they take too long. This is not a bug, the tests are completing successfully -- however they are designed to cause serv...
by tfaber＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: tfaber Date: Sun Feb 15 18:20:59 2015 New Revision: 66302 URL: http://svn.reactos.org/svn/reactos?rev=66302&view=rev Log: [ADVAPI32_WINETEST] - Skip over service timeout tests because they take too long. This is not a bug, the tests are completing successfully -- however they are designed to cause service manager timeouts and take two minutes in doing so. Testbot time is too valuable to do this on every commit. ROSTESTS-151 Modified: trunk/rostests/winetests/advapi32/service.c Modified: trunk/rostests/winetests/advapi32/service.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/advapi32/servic… ============================================================================== --- trunk/rostests/winetests/advapi32/service.c [iso-8859-1] (original) +++ trunk/rostests/winetests/advapi32/service.c [iso-8859-1] Sun Feb 15 18:20:59 2015 @@ -2266,6 +2266,12 @@ goto cleanup; } + if (!winetest_interactive) + { + skip("ROSTESTS-151: Skipping service start timeout tests because they take too long. This is not a bug!\n"); + goto cleanup; + } + /* Again with a process that exits right away */ displayname = "Winetest Exit Service"; ret = ChangeServiceConfigA(svc_handle, SERVICE_NO_CHANGE, SERVICE_NO_CHANGE, SERVICE_NO_CHANGE, cmd, NULL, NULL, NULL, NULL, NULL, displayname);

1 0

[dquintana] 66301: [SHELL32] * Include enumeration of hidden files in the default shell view, until we have a proper way of toggling it. CORE-9192
by dquintana＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: dquintana Date: Sun Feb 15 18:05:34 2015 New Revision: 66301 URL: http://svn.reactos.org/svn/reactos?rev=66301&view=rev Log: [SHELL32] * Include enumeration of hidden files in the default shell view, until we have a proper way of toggling it. CORE-9192 Modified: trunk/reactos/dll/win32/shell32/CDefView.cpp Modified: trunk/reactos/dll/win32/shell32/CDefView.cpp URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/shell32/CDefView… ============================================================================== --- trunk/reactos/dll/win32/shell32/CDefView.cpp [iso-8859-1] (original) +++ trunk/reactos/dll/win32/shell32/CDefView.cpp [iso-8859-1] Sun Feb 15 18:05:34 2015 @@ -868,7 +868,8 @@ TRACE("%p\n", this); /* get the itemlist from the shfolder*/ - hRes = m_pSFParent->EnumObjects(m_hWnd, SHCONTF_NONFOLDERS | SHCONTF_FOLDERS, &pEnumIDList); + /* FIXME: make showing hidden files a setting. */ + hRes = m_pSFParent->EnumObjects(m_hWnd, SHCONTF_NONFOLDERS | SHCONTF_FOLDERS | SHCONTF_INCLUDEHIDDEN, &pEnumIDList); if (hRes != S_OK) { if (hRes == S_FALSE)

1 0

[tfaber] 66300: [TDILIB][IPHLPAPI][WSHTCPIP] - Request only the access rights that are actually required in openTcpFile. Fixes many should-be access denied errors originating from netshell. While t...
by tfaber＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: tfaber Date: Sun Feb 15 17:57:34 2015 New Revision: 66300 URL: http://svn.reactos.org/svn/reactos?rev=66300&view=rev Log: [TDILIB][IPHLPAPI][WSHTCPIP] - Request only the access rights that are actually required in openTcpFile. Fixes many should-be access denied errors originating from netshell. While this doesn't fix the root cause of CORE-9172, it is a required step if we ever want netshell to work for non-Admin users. CORE-9172 Modified: trunk/reactos/dll/win32/iphlpapi/address.c trunk/reactos/dll/win32/iphlpapi/ifenum_reactos.c trunk/reactos/dll/win32/iphlpapi/iphlpapi_main.c trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c trunk/reactos/dll/win32/iphlpapi/route_reactos.c trunk/reactos/dll/win32/wshtcpip/wshtcpip.c trunk/reactos/lib/tdilib/enum.c trunk/reactos/lib/tdilib/handle.c trunk/reactos/lib/tdilib/tdilib.h Modified: trunk/reactos/dll/win32/iphlpapi/address.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/address… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/address.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/address.c [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -306,7 +306,7 @@ ZeroMemory(Ptr, RemainingSize); /* open the tcpip driver */ - Status = openTcpFile(&TcpFile); + Status = openTcpFile(&TcpFile, FILE_READ_DATA); if (!NT_SUCCESS(Status)) { ERR("Could not open handle to tcpip.sys. Status %08x\n", Status); Modified: trunk/reactos/dll/win32/iphlpapi/ifenum_reactos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/ifenum_… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/ifenum_reactos.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/ifenum_reactos.c [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -225,7 +225,7 @@ NTSTATUS status; int i; - status = openTcpFile( &tcpFile ); + status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( !NT_SUCCESS(status) ) { WARN("getNumInterfaces: failed %08x\n", status ); @@ -350,7 +350,7 @@ IFInfo ifInfo; HANDLE tcpFile; char *interfaceName = 0, *adapter_name = 0; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( NT_SUCCESS(status) ) { status = getInterfaceInfoByIndex( tcpFile, index, &ifInfo ); @@ -379,7 +379,7 @@ { IFInfo ifInfo; HANDLE tcpFile; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( NT_SUCCESS(status) ) { status = getInterfaceInfoByName( tcpFile, (char *)name, &ifInfo ); @@ -400,7 +400,7 @@ IFInfo *ifInfo; InterfaceIndexTable *ret = 0; HANDLE tcpFile; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( NT_SUCCESS(status) ) { status = getInterfaceInfoSet( tcpFile, &ifInfo, &numInterfaces ); @@ -476,7 +476,7 @@ NTSTATUS status = STATUS_SUCCESS; DWORD addrOut = INADDR_ANY; - status = openTcpFile( &tcpFile ); + status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( NT_SUCCESS(status) ) { status = getIPAddrEntryForIf( tcpFile, name, index, &ifInfo ); @@ -527,7 +527,7 @@ { HANDLE tcpFile; IFInfo info; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( NT_SUCCESS(status) ) { status = getInterfaceInfoByName( tcpFile, (char *)name, &info ); @@ -544,7 +544,7 @@ { HANDLE tcpFile; IFInfo info; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( NT_SUCCESS(status) ) { status = getInterfaceInfoByIndex( tcpFile, index, &info ); @@ -581,7 +581,7 @@ { HANDLE tcpFile; IFInfo info; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA ); TRACE("Called.\n"); @@ -606,7 +606,7 @@ { HANDLE tcpFile; IFInfo info; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA ); TRACE("Called.\n"); @@ -641,7 +641,7 @@ PULONG NteContext, PULONG NteInstance ) { HANDLE tcpFile; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA | FILE_WRITE_DATA ); IP_SET_DATA Data; IO_STATUS_BLOCK Iosb; @@ -682,7 +682,7 @@ NTSTATUS deleteIpAddress( ULONG NteContext ) { HANDLE tcpFile; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA | FILE_WRITE_DATA ); IO_STATUS_BLOCK Iosb; TRACE("Called.\n"); Modified: trunk/reactos/dll/win32/iphlpapi/iphlpapi_main.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/iphlpap… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/iphlpapi_main.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/iphlpapi_main.c [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -2145,7 +2145,7 @@ if (!pArpEntry) return ERROR_INVALID_PARAMETER; - if (!NT_SUCCESS(openTcpFile( &tcpFile ))) + if (!NT_SUCCESS(openTcpFile( &tcpFile, FILE_READ_DATA | FILE_WRITE_DATA ))) return ERROR_NOT_SUPPORTED; if (!NT_SUCCESS(getNthIpEntity( tcpFile, pArpEntry->dwIndex, &id ))) @@ -2312,7 +2312,7 @@ if (!indexTable) return ERROR_NOT_ENOUGH_MEMORY; - ret = openTcpFile(&tcpFile); + ret = openTcpFile(&tcpFile, FILE_READ_DATA); if (!NT_SUCCESS(ret)) return ERROR_NO_DATA; Modified: trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/ipstats… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/ipstats_reactos.c [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -366,7 +366,7 @@ TRACE("called.\n"); - status = openTcpFile( &tcpFile ); + status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( !NT_SUCCESS(status) ) { TRACE("failure: %08x\n", (int)status ); @@ -421,7 +421,7 @@ DWORD numRoutes = getNumRoutes(), routesAdded = 0; TDIEntityID ent; HANDLE tcpFile; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA ); int i; if( !NT_SUCCESS(status) ) @@ -498,7 +498,7 @@ TRACE("called.\n"); - status = openTcpFile( &tcpFile ); + status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( !NT_SUCCESS(status) ) { TRACE("failure: %08x\n", (int)status ); @@ -550,7 +550,7 @@ totalNumber = getNumArpEntries(); - status = openTcpFile( &tcpFile ); + status = openTcpFile( &tcpFile, FILE_READ_DATA ); if( !NT_SUCCESS(status) ) { TRACE("failure: %08x\n", (int)status ); Modified: trunk/reactos/dll/win32/iphlpapi/route_reactos.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/iphlpapi/route_r… ============================================================================== --- trunk/reactos/dll/win32/iphlpapi/route_reactos.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/iphlpapi/route_reactos.c [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -31,7 +31,7 @@ DWORD createIpForwardEntry( PMIB_IPFORWARDROW pRoute ) { HANDLE tcpFile; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA | FILE_WRITE_DATA ); TCP_REQUEST_SET_INFORMATION_EX_ROUTE_ENTRY req = TCP_REQUEST_SET_INFORMATION_INIT; IPRouteEntry *rte; @@ -96,7 +96,7 @@ DWORD deleteIpForwardEntry( PMIB_IPFORWARDROW pRoute ) { HANDLE tcpFile; - NTSTATUS status = openTcpFile( &tcpFile ); + NTSTATUS status = openTcpFile( &tcpFile, FILE_READ_DATA | FILE_WRITE_DATA ); TCP_REQUEST_SET_INFORMATION_EX_ROUTE_ENTRY req = TCP_REQUEST_SET_INFORMATION_INIT; IPRouteEntry *rte; Modified: trunk/reactos/dll/win32/wshtcpip/wshtcpip.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/dll/win32/wshtcpip/wshtcpi… ============================================================================== --- trunk/reactos/dll/win32/wshtcpip/wshtcpip.c [iso-8859-1] (original) +++ trunk/reactos/dll/win32/wshtcpip/wshtcpip.c [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -391,7 +391,7 @@ HANDLE TcpCC; DWORD BytesReturned; - if (openTcpFile(&TcpCC) != STATUS_SUCCESS) + if (openTcpFile(&TcpCC, FILE_READ_DATA | FILE_WRITE_DATA) != STATUS_SUCCESS) return WSAEINVAL; Status = DeviceIoControl(TcpCC, @@ -449,7 +449,7 @@ case WSH_NOTIFY_BIND: DPRINT("WSHNotify: WSH_NOTIFY_BIND\n"); - Status = openTcpFile(&TcpCC); + Status = openTcpFile(&TcpCC, FILE_READ_DATA); if (Status != STATUS_SUCCESS) return WSAEINVAL; Modified: trunk/reactos/lib/tdilib/enum.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/tdilib/enum.c?rev=6630… ============================================================================== --- trunk/reactos/lib/tdilib/enum.c [iso-8859-1] (original) +++ trunk/reactos/lib/tdilib/enum.c [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -26,7 +26,7 @@ DWORD toiType, DWORD toiId, DWORD teiEntity, - DWORD teiInstance, + DWORD teiInstance, DWORD fixedPart, DWORD entrySize, PVOID *tdiEntitySet, @@ -113,7 +113,7 @@ INFO_TYPE_PROVIDER, ENTITY_LIST_ID, GENERIC_ENTITY, - 0, + 0, 0, sizeof(TDIEntityID), (PVOID *)entitySet, Modified: trunk/reactos/lib/tdilib/handle.c URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/tdilib/handle.c?rev=66… ============================================================================== --- trunk/reactos/lib/tdilib/handle.c [iso-8859-1] (original) +++ trunk/reactos/lib/tdilib/handle.c [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -9,7 +9,7 @@ const PWCHAR TcpFileName = L"\\Device\\Tcp"; -NTSTATUS openTcpFile(PHANDLE tcpFile) +NTSTATUS openTcpFile(PHANDLE tcpFile, ACCESS_MASK DesiredAccess) { UNICODE_STRING fileName; OBJECT_ATTRIBUTES objectAttributes; @@ -24,18 +24,12 @@ NULL, NULL ); - status = ZwCreateFile( tcpFile, - SYNCHRONIZE | GENERIC_EXECUTE | - GENERIC_READ | GENERIC_WRITE, - &objectAttributes, - &ioStatusBlock, - NULL, - FILE_ATTRIBUTE_NORMAL, - FILE_SHARE_READ | FILE_SHARE_WRITE, - FILE_OPEN_IF, - FILE_SYNCHRONOUS_IO_NONALERT, - 0, - 0 ); + status = NtOpenFile( tcpFile, + DesiredAccess | SYNCHRONIZE, + &objectAttributes, + &ioStatusBlock, + FILE_SHARE_READ | FILE_SHARE_WRITE, + FILE_SYNCHRONOUS_IO_NONALERT); /* String does not need to be freed: it points to the constant * string we provided */ Modified: trunk/reactos/lib/tdilib/tdilib.h URL: http://svn.reactos.org/svn/reactos/trunk/reactos/lib/tdilib/tdilib.h?rev=66… ============================================================================== --- trunk/reactos/lib/tdilib/tdilib.h [iso-8859-1] (original) +++ trunk/reactos/lib/tdilib/tdilib.h [iso-8859-1] Sun Feb 15 17:57:34 2015 @@ -7,7 +7,7 @@ #pragma once -NTSTATUS openTcpFile(PHANDLE tcpFile); +NTSTATUS openTcpFile(PHANDLE tcpFile, ACCESS_MASK DesiredAccess); VOID closeTcpFile(HANDLE tcpFile); NTSTATUS tdiGetEntityIDSet( HANDLE tcpFile, TDIEntityID **entitySet, PDWORD numEntities );

1 0

[tfaber] 66299: [OLEAUT32_WINETEST] - Re-enable test_apm and test_enhmetafile. ROSTESTS-2
by tfaber＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: tfaber Date: Sun Feb 15 17:54:41 2015 New Revision: 66299 URL: http://svn.reactos.org/svn/reactos?rev=66299&view=rev Log: [OLEAUT32_WINETEST] - Re-enable test_apm and test_enhmetafile. ROSTESTS-2 Modified: trunk/rostests/winetests/oleaut32/olepicture.c Modified: trunk/rostests/winetests/oleaut32/olepicture.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/oleaut32/olepic… ============================================================================== --- trunk/rostests/winetests/oleaut32/olepicture.c [iso-8859-1] (original) +++ trunk/rostests/winetests/oleaut32/olepicture.c [iso-8859-1] Sun Feb 15 17:54:41 2015 @@ -564,11 +564,6 @@ BOOL keep; short type; - if(!winetest_interactive) { - skip("ROSTESTS-2: oleaut_winetest:olepicture crashes with Page Fault.\n"); - return; - } - hglob = GlobalAlloc (0, sizeof(apmdata)); data = GlobalLock(hglob); memcpy(data, apmdata, sizeof(apmdata)); @@ -624,11 +619,6 @@ LONG cxy; BOOL keep; short type; - - if(!winetest_interactive) { - skip("ROSTESTS-2: oleaut_winetest:olepicture crashes with Page Fault.\n"); - return; - } hglob = GlobalAlloc (0, sizeof(enhmetafile)); data = GlobalLock(hglob);

1 0

[tfaber] 66298: [URLMON_WINETEST] - Re-enable protocol test. ROSTESTS-38
by tfaber＠svn.reactos.org 15 Feb '15

15 Feb '15

Author: tfaber Date: Sun Feb 15 17:50:18 2015 New Revision: 66298 URL: http://svn.reactos.org/svn/reactos?rev=66298&view=rev Log: [URLMON_WINETEST] - Re-enable protocol test. ROSTESTS-38 Modified: trunk/rostests/winetests/urlmon/protocol.c Modified: trunk/rostests/winetests/urlmon/protocol.c URL: http://svn.reactos.org/svn/reactos/trunk/rostests/winetests/urlmon/protocol… ============================================================================== --- trunk/rostests/winetests/urlmon/protocol.c [iso-8859-1] (original) +++ trunk/rostests/winetests/urlmon/protocol.c [iso-8859-1] Sun Feb 15 17:50:18 2015 @@ -3748,12 +3748,6 @@ { HMODULE hurlmon; - if(!winetest_interactive) - { - skip("protocol test skipped, ROSTESTS-38\n"); - return; - } - hurlmon = GetModuleHandleA("urlmon.dll"); pCoInternetGetSession = (void*) GetProcAddress(hurlmon, "CoInternetGetSession"); pReleaseBindInfo = (void*) GetProcAddress(hurlmon, "ReleaseBindInfo");

1 0

← Newer
1
...
18
19
20
21
22
23
24
...
38
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs February 2015