[ekohl] 58391: [WINLOGON] - Replace the old LogonStatus variable by a LOGON_STATE enum type. - Declare and document all new states and their transitions.
2 Mar
2013
2 Mar
'13
11:58 a.m.
Author: ekohl
Date: Sat Mar 2 11:58:26 2013
New Revision: 58391
URL: http://svn.reactos.org/svn/reactos?rev=58391&view=rev
Log:
[WINLOGON]
- Replace the old LogonStatus variable by a LOGON_STATE enum type.
- Declare and document all new states and their transitions.
Modified:
trunk/reactos/base/system/winlogon/sas.c
trunk/reactos/base/system/winlogon/winlogon.c
trunk/reactos/base/system/winlogon/winlogon.h
Modified: trunk/reactos/base/system/winlogon/sas.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/winlogon/sas.c…
==============================================================================
--- trunk/reactos/base/system/winlogon/sas.c [iso-8859-1] (original)
+++ trunk/reactos/base/system/winlogon/sas.c [iso-8859-1] Sat Mar 2 11:58:26 2013
@@ -700,7 +700,7 @@
UnloadUserProfile(Session->UserToken, Session->hProfileInfo);
CloseHandle(Session->UserToken);
UpdatePerUserSystemParameters(0, FALSE);
- Session->LogonStatus = WKSTA_IS_LOGGED_OFF;
+ Session->LogonState = STATE_LOGGED_OFF;
Session->UserToken = NULL;
return STATUS_SUCCESS;
}
@@ -828,7 +828,7 @@
if (HandleLogon(Session))
{
SwitchDesktop(Session->ApplicationDesktop);
- Session->LogonStatus = WKSTA_IS_LOGGED_ON;
+ Session->LogonState = STATE_LOGGED_ON;
}
else
Session->Gina.Functions.WlxDisplaySASNotice(Session->Gina.Context);
@@ -839,7 +839,7 @@
if (Session->Gina.Functions.WlxIsLockOk(Session->Gina.Context))
{
SwitchDesktop(WLSession->WinlogonDesktop);
- Session->LogonStatus = WKSTA_IS_LOCKED;
+ Session->LogonState = STATE_LOCKED;
Session->Gina.Functions.WlxDisplayLockedNotice(Session->Gina.Context);
}
break;
@@ -847,7 +847,7 @@
case WLX_SAS_ACTION_SHUTDOWN: /* 0x05 */
case WLX_SAS_ACTION_SHUTDOWN_POWER_OFF: /* 0x0a */
case WLX_SAS_ACTION_SHUTDOWN_REBOOT: /* 0x0b */
- if (Session->LogonStatus != WKSTA_IS_LOGGED_OFF)
+ if (Session->LogonState != STATE_LOGGED_OFF)
{
if (!Session->Gina.Functions.WlxIsLogoffOk(Session->Gina.Context))
break;
@@ -880,7 +880,7 @@
break;
case WLX_SAS_ACTION_UNLOCK_WKSTA: /* 0x08 */
SwitchDesktop(WLSession->ApplicationDesktop);
- Session->LogonStatus = WKSTA_IS_LOGGED_ON;
+ Session->LogonState = STATE_LOGGED_ON;
break;
default:
WARN("Unknown SAS action 0x%lx\n", wlxAction);
@@ -894,9 +894,9 @@
{
DWORD wlxAction = WLX_SAS_ACTION_NONE;
- if (Session->LogonStatus == WKSTA_IS_LOGGED_ON)
+ if (Session->LogonState == STATE_LOGGED_ON)
wlxAction = (DWORD)Session->Gina.Functions.WlxLoggedOnSAS(Session->Gina.Context,
dwSasType, NULL);
- else if (Session->LogonStatus == WKSTA_IS_LOCKED)
+ else if (Session->LogonState == STATE_LOCKED)
wlxAction =
(DWORD)Session->Gina.Functions.WlxWkstaLockedSAS(Session->Gina.Context, dwSasType);
else
{
Modified: trunk/reactos/base/system/winlogon/winlogon.c
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/winlogon/winlo…
==============================================================================
--- trunk/reactos/base/system/winlogon/winlogon.c [iso-8859-1] (original)
+++ trunk/reactos/base/system/winlogon/winlogon.c [iso-8859-1] Sat Mar 2 11:58:26 2013
@@ -412,7 +412,7 @@
//DisplayStatusMessage(Session, Session->WinlogonDesktop,
IDS_APPLYINGCOMPUTERSETTINGS);
/* Display logged out screen */
- WLSession->LogonStatus = WKSTA_IS_LOGGED_OFF;
+ WLSession->LogonState = STATE_LOGGED_OFF;
RemoveStatusMessage(WLSession);
/* Check for pending setup */
Modified: trunk/reactos/base/system/winlogon/winlogon.h
URL:
http://svn.reactos.org/svn/reactos/trunk/reactos/base/system/winlogon/winlo…
==============================================================================
--- trunk/reactos/base/system/winlogon/winlogon.h [iso-8859-1] (original)
+++ trunk/reactos/base/system/winlogon/winlogon.h [iso-8859-1] Sat Mar 2 11:58:26 2013
@@ -119,11 +119,87 @@
BOOL UseCtrlAltDelete;
} GINAINSTANCE, *PGINAINSTANCE;
-/* FIXME: put in an enum */
-/* See
http://msdn.microsoft.com/library/default.asp?url=/library/en-us/secauthn/s…
*/
-#define WKSTA_IS_LOGGED_OFF 0
-#define WKSTA_IS_LOGGED_ON 1
-#define WKSTA_IS_LOCKED 2
+
+/*
+ * The picture Microfoft is trying to paint here
+ * (http://msdn.microsoft.com/en-us/library/windows/desktop/aa380547%28v=vs.85%…)
+ * about the Winlogon states is a little too simple.
+ *
+ * The real picture should look more like this:
+ *
+ * STATE_INIT
+ * Initial state. Required for session initialization. After intialization,
+ * the state will automatically change to STATE_LOGGED_OFF.
+ *
+ * STATE_LOGGED_OFF
+ * User is logged of. Winlogon shows the "Press Ctrl-Alt-Del for logon"
+ * dialog. The state changes to STATE_LOGGED_OFF_SAS when the user presses
+ * "Ctrl-Alt-Del". If DisableCAD is true, the state will automatically
+ * change to STATE_LOGGED_OFF_SAS without showing the dialog.
+ *
+ * STATE_LOGGED_OFF_SAS
+ * State shows the logon dialog. Entering the right credentials an pressing
+ * "OK" changes the state to STATE_LOGGED_ON. Pressing "Cancel" or
a Timeout
+ * changes the state back to STATE_LOGGED_OFF.
+ *
+ * STATE_LOGGED_ON
+ * User is logged on. Winlogon does not show any dialog. Pressing
+ * "Ctrl-Alt-Del" changes the state to STATE_LOGGED_ON_SAS and user
+ * inactivity changes the state to STATE_SCREENSAVER.
+ *
+ * STATE_LOGGED_ON_SAS
+ * Winlogon shows the security dialog. Pressing "Cancel" or "Task
Manager"
+ * or a Timeout change the state back to STATE_LOGGED_ON. Pressing "Change
+ * Password" does not change the state, because the security dialog is still
+ * visible behind the change password dialog. Pressing "Log off" changes
the
+ * state to STATE_LOGGING_OFF. Pressing "Lock Computer" changes the state
to
+ * STATE_LOCKED. Pressing "Shutdown" changes the state to
+ * STATE_SUUTTING_DOWN.
+ *
+ * STATE_SCREENSAVER
+ * Winlogon runs the screen saver. Upon user activity, the screensaver
+ * terminates and the state changes back to STATE_LOGGED_ON if the secure
+ * screen saver option is off. Otherwise, the state changes to STATE_LOCKED.
+ *
+ * STATE_LOGGING_OFF
+ * Winlogon shows the logoff dialog. Pressing "Cancel" or a timeout changes
+ * the state back to STATE_LOGGED_ON_SAS. Pressing "OK" logs off the user
+ * and changes the state to STATE_LOGGED_OFF.
+ *
+ * STATE_LOCKED
+ * Winlogon shows the locked message dialog. When the user presses "Ctrl-
+ * Alt-Del" the state changes to STATE_LOCKED_SAS. If DisableCAD is true,
+ * the state will automatically change to STATE_LOCKED_SAS without showing
+ * the dialog.
+ *
+ * STATE_LOCKED_SAS
+ * Winlogon shows the unlock dialog. Presing "Cancel" or a timeout will
+ * change the state back to STATE_LOCKED. Entering the right credentials and
+ * pressing "OK" unlocks the computer and changes the state to
+ * STATE_LOGGED_ON.
+ *
+ * STATE_SHUTTING_DOWN
+ * Winlogon shows the shutdown dialog. Presing "Cancel" or a timeout will
+ * change the state back to STATE_LOGGED_ON_SAS. Pressing "OK" will change
+ * the state to STATE_SHUT_DOWN.
+ *
+ * STATE_SHUT_DOWN
+ * Terminates winlogon and initiates shut-down.
+ */
+typedef enum _LOGON_STATE
+{
+ STATE_INIT, // not user yet
+ STATE_LOGGED_OFF,
+ STATE_LOGGED_OFF_SAS, // not user yet
+ STATE_LOGGED_ON,
+ STATE_LOGGED_ON_SAS, // not user yet
+ STATE_SCREENSAVER, // not user yet
+ STATE_LOCKED,
+ STATE_LOCKED_SAS, // not user yet
+ STATE_LOGGING_OFF, // not user yet
+ STATE_SHUTTING_DOWN, // not user yet
+ STATE_SHUT_DOWN // not user yet
+} LOGON_STATE, *PLOGON_STATE;
#define LockWorkstation(Session)
#define UnlockWorkstation(Session)
@@ -143,7 +219,7 @@
LUID LogonId;
HANDLE UserToken;
HANDLE hProfileInfo;
- DWORD LogonStatus;
+ LOGON_STATE LogonState;
DWORD DialogTimeout; /* Timeout for dialog boxes, in seconds */
/* Screen-saver informations */
4275
days inactive
4275
days old
0 comments
1 participants
participants (1)
-
ekohl＠svn.reactos.org