X-Git-Url: https://git.mxchange.org/?a=blobdiff_plain;f=inc%2Fmodules%2Fadmin.php;h=0a6951b998c4a3147572530a631349d905dc63cd;hb=7ce379f8009cb988e0af74c5e5bf6f8ddb6de37b;hp=515c745ee9fb0e8ffc28c647ac18bc9e0b2e8f2a;hpb=dbeb4ef1dc2e59d0c6bfab9d9666cdfa515f58e7;p=mailer.git

diff --git a/inc/modules/admin.php b/inc/modules/admin.php
index 515c745ee9..0a6951b998 100644
--- a/inc/modules/admin.php
+++ b/inc/modules/admin.php
@@ -41,7 +41,7 @@ if (!defined('__SECURITY')) {
 if ((empty($GLOBALS['action'])) && ($check == "admin_only")) {
 	// Redirect to right URL
 	LOAD_URL("modules.php?module=admin&action=login");
-}
+} // END - if
 
 // Load include file
 require_once(PATH."inc/modules/admin/admin-inc.php");
@@ -90,14 +90,14 @@ if (!isBooleanConstantAndTrue('admin_registered')) {
 				$ret = "done";
 			}
 
+			// Admin still not registered?
 			if (!isBooleanConstantAndTrue('admin_registered')) {
 				// Write to config that registration is done
 				admin_WriteData(PATH."inc/config.php", "ADMIN-SETUP", "define('admin_registered', ", ");", "true", 0);
 
 				// Load URL for login
-				$URL = URL."/modules.php?module=admin&action=login";
-				LOAD_URL($URL);
-			}
+				LOAD_URL(URL."/modules.php?module=admin&action=login");
+			} // END - if
 			break;
 		}
 	}
@@ -189,9 +189,9 @@ if (!isBooleanConstantAndTrue('admin_registered')) {
 	// At leat one administrator account was created
 	if ((isSessionVariableSet('admin_login')) && (isSessionVariableSet('admin_md5')) && (isSessionVariableSet('admin_last')) && (isSessionVariableSet('admin_to'))) {
 		// Timeout for last login, we have to logout first!
-		$URL = URL."/modules.php?module=admin&action=login&logout=1";
-		LOAD_URL($URL);
-	}
+		LOAD_URL(URL."/modules.php?module=admin&action=login&logout=1");
+	} // END - if
+
 	if (!empty($_GET['register'])) {
 		// Registration of first admin is done
 		if ($_GET['register'] == "done") OUTPUT_HTML("<STRONG class=\"admin\">".ADMIN_REGISTER_DONE."</STRONG>");
@@ -214,7 +214,7 @@ if (!isBooleanConstantAndTrue('admin_registered')) {
 			// Rewrite overview module
 			if ($GLOBALS['what'] == "overview") {
 				$GLOBALS['action'] = GET_ACTION($GLOBALS['module'], $GLOBALS['what']);
-			}
+			} // END - if
 
 			// Add data to URL
 			if (!empty($GLOBALS['what'])) $URL .= "what=".$GLOBALS['what'];
@@ -228,14 +228,18 @@ if (!isBooleanConstantAndTrue('admin_registered')) {
 		case "404": // Administrator login not found
 			$_POST['ok'] = $ret;
 			$ret = ADMIN_NOT_FOUND;
+			DESTROY_ADMIN_SESSION();
 			break;
 
 		case "pass": // Wrong password
 			$_POST['ok'] = $ret;
 			$ret = WRONG_PASS." [<A href=\"".URL."/modules.php?module=admin&amp;reset_pass=1\">".ADMIN_RESET_PASS."</A>]\n";
+			DESTROY_ADMIN_SESSION();
 			break;
-		}
-	}
+		} // END - switch
+	} // END - if
+
+	// Error detected?
 	if ($ret != "done") {
 		if (!empty($_POST['login'])) {
 			define('__LOGIN_VALUE', $_POST['login']);
@@ -245,8 +249,8 @@ if (!isBooleanConstantAndTrue('admin_registered')) {
 
 		if (isset($_POST['ok'])) {
 			// Set messages to zero
-
 			$MSG1 = ""; $MSG2 = "";
+
 			// No login entered?
 			if (empty($_POST['login'])) $MSG1 = ADMIN_NO_LOGIN;
 
@@ -296,19 +300,10 @@ if (!isBooleanConstantAndTrue('admin_registered')) {
 
 		// Load login form template
 		LOAD_TEMPLATE("admin_login_form", false, $content);
-	}
+	} // END - if
 } elseif (isset($_GET['logout'])) {
 	// Only try to remove cookies
-	if (set_session("admin_login", "") && set_session("admin_md5", "") && set_session("admin_last", "") && set_session("admin_to", "")) {
-		// Also remove array elements
-		set_session('admin_login', "");
-		set_session('admin_md5'  , "");
-		set_session('admin_last' , "");
-		set_session('admin_to'   , "");
-
-		// Destroy session
-		@session_destroy();
-
+	if (DESTROY_ADMIN_SESSION()) {
 		// Load logout template
 		if (isset($_GET['register'])) {
 			// Secure input
@@ -387,11 +382,13 @@ if (!isBooleanConstantAndTrue('admin_registered')) {
 
 	case "404": // Administrator login not found
 		$_POST['ok'] = $ret;
+		DESTROY_ADMIN_SESSION();
 		ADD_FATAL(ADMIN_NOT_FOUND);
 		break;
 
 	case "pass": // Wrong password
 		$_POST['ok'] = $ret;
+		DESTROY_ADMIN_SESSION();
 		ADD_FATAL(WRONG_PASS);
 		break;
 	}