Fix forgotten password vulnerability

Fix forgotten password vulnerability
This commit is contained in:
novgorodschi catalin
2026-07-03 10:07:38 +03:00
parent b24033f19a
commit 5f005e1d56
+132 -77
View File
@@ -3,10 +3,10 @@
################################################################################# #################################################################################
## -= YOU MAY NOT REMOVE OR CHANGE THIS NOTICE =- ## ## -= YOU MAY NOT REMOVE OR CHANGE THIS NOTICE =- ##
## --------------------------------------------------------------------------- ## ## --------------------------------------------------------------------------- ##
## Filename : password.php ## ## Filename : password.php ##
## Type : In Game Password Page ## ## Type : In Game Password Page ##
## --------------------------------------------------------------------------- ## ## --------------------------------------------------------------------------- ##
## Developed by : Dzoki ## ## Developed by : Dzoki ##
## Refactored by : Shadow ## ## Refactored by : Shadow ##
## Redesign by : Shadow ## ## Redesign by : Shadow ##
## --------------------------------------------------------------------------- ## ## --------------------------------------------------------------------------- ##
@@ -22,118 +22,173 @@
use App\Utils\AccessLogger; use App\Utils\AccessLogger;
if(!file_exists('var/installed') && @opendir('install')) { if (!file_exists('var/installed') && @opendir('install')) {
header("Location: install/"); header("Location: install/");
exit; exit;
} }
include_once("GameEngine/config.php"); include_once("GameEngine/config.php");
include_once("GameEngine/Lang/" . LANG . ".php"); include_once("GameEngine/Lang/" . LANG . ".php");
include_once("GameEngine/Database.php"); include_once("GameEngine/Database.php");
include_once("GameEngine/Mailer.php"); include_once("GameEngine/Mailer.php");
include_once("GameEngine/Generator.php"); include_once("GameEngine/Generator.php");
AccessLogger::logRequest(); AccessLogger::logRequest();
if(!isset($_REQUEST['npw'])){ if (!isset($_REQUEST['npw'])) {
header("Location: login.php"); header("Location: login.php");
exit; exit;
} }
?> ?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd"> <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN"
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html> <html>
<head> <head>
<title><?php echo SERVER_NAME; ?> - Forgotten Password</title> <title><?php echo SERVER_NAME; ?> - Forgotten Password</title>
<link rel="shortcut icon" href="favicon.ico"/> <link rel="shortcut icon" href="favicon.ico"/>
<meta name="content-language" content="en" /> <meta name="content-language" content="en" />
<meta http-equiv="cache-control" content="max-age=0" /> <meta http-equiv="cache-control" content="max-age=0" />
<meta http-equiv="imagetoolbar" content="no" /> <meta http-equiv="imagetoolbar" content="no" />
<meta http-equiv="content-type" content="text/html; charset=UTF-8" /> <meta http-equiv="content-type" content="text/html; charset=UTF-8" />
<script src="mt-core.js?0faab" type="text/javascript"></script>
<script src="mt-more.js?0faab" type="text/javascript"></script> <script src="mt-core.js?0faab" type="text/javascript"></script>
<script src="unx.js?f4b7h" type="text/javascript"></script> <script src="mt-more.js?0faab" type="text/javascript"></script>
<script src="new.js?0faab" type="text/javascript"></script> <script src="unx.js?f4b7h" type="text/javascript"></script>
<link href="<?php echo GP_LOCATE; ?>lang/en/compact.css?f4b7i" rel="stylesheet" type="text/css" /> <script src="new.js?0faab" type="text/javascript"></script>
<link href="<?php echo GP_LOCATE; ?>lang/en/lang.css?f4b7d" rel="stylesheet" type="text/css" />
<link href="<?php echo GP_LOCATE ?>travian.css?f4b7d" rel="stylesheet" type="text/css" /> <link href="<?php echo GP_LOCATE; ?>lang/en/compact.css?f4b7i" rel="stylesheet" type="text/css" />
<link href="<?php echo GP_LOCATE ?>lang/en/lang.css" rel="stylesheet" type="text/css" /> <link href="<?php echo GP_LOCATE; ?>lang/en/lang.css?f4b7d" rel="stylesheet" type="text/css" />
</head> <link href="<?php echo GP_LOCATE; ?>travian.css?f4b7d" rel="stylesheet" type="text/css" />
<link href="<?php echo GP_LOCATE; ?>lang/en/lang.css" rel="stylesheet" type="text/css" />
</head>
<body class="v35 ie ie7" onload="initCounter()"> <body class="v35 ie ie7" onload="initCounter()">
<div class="wrapper"> <div class="wrapper">
<div id="dynamic_header"> <div id="dynamic_header"></div>
</div>
<div id="header"></div> <div id="header"></div>
<div id="mid"> <div id="mid">
<?php include("Templates/menu.tpl"); ?> <?php include("Templates/menu.tpl"); ?>
<div id="content" class="activate">
<h1><img src="img/x.gif" class="passwort" alt="new password" /></h1> <div id="content" class="activate">
<h5><img src="img/x.gif" class="img_u22" alt="forgotten password" /></h5>
<h1><img src="img/x.gif" class="passwort" alt="new password" /></h1>
<h5><img src="img/x.gif" class="img_u22" alt="forgotten password" /></h5>
<?php <?php
// user input email and submit
if(isset($_POST['email']) && isset($_POST['npw'])){
$uid = intval($_POST['npw']);
$email = $database->getUserField($uid, 'email', 0);
$username = $database->getUserField($uid, 'username', 0);
if($email != $_POST['email']){
echo "<p>Unfortunately the entered email address does not match the one used to register the account.</p>\n";
}else{
// generate password and cpw
$npw = $generator->generateRandStr(7);
$cpw = $generator->generateRandStr(10);
$database->addPassword($uid, $npw, $cpw); // User submitted email address
if (isset($_POST['email']) && isset($_POST['npw'])) {
// send password mail $uid = (int)$_POST['npw'];
$mailer->sendPassword($email, $uid, $username, $npw, $cpw); $submittedEmail = trim($_POST['email']);
echo "<p>Password was sent to: ${_POST['email']}</p>\n"; // Mesaj generic pentru a evita user enumeration
} $genericMessage =
'<p>If the account information is valid, a password reset email has been sent.</p>';
// user click the link in 'password forgotten' email if ($uid <= 0 || $submittedEmail === '') {
}else if(isset($_GET['cpw']) && isset($_GET['npw'])){
$uid = intval($_GET['npw']);
$cpw = preg_replace('#[^a-zA-Z0-9]#', '', $_GET['cpw']);
if(!$database->resetPassword($uid, $cpw)){ echo $genericMessage;
echo '<p>The password has not been changed. Perhaps the activation code has already been used.</p>';
}else{
echo '<p>The password has been successfully changed.</p>';
}
} else {
// user click 'generate password' link in login fail page, display input form here $email = $database->getUserField($uid, 'email', 0);
}else { $username = $database->getUserField($uid, 'username', 0);
// Verifică dacă utilizatorul există și emailul corespunde
if (
!is_string($email) ||
$email === '' ||
!is_string($username) ||
$username === '' ||
!hash_equals($email, $submittedEmail)
) {
echo $genericMessage;
} else {
// Generate password and confirmation code
$npw = $generator->generateRandStr(7);
$cpw = $generator->generateRandStr(10);
$database->addPassword($uid, $npw, $cpw);
// Send password email
$mailer->sendPassword($email, $uid, $username, $npw, $cpw);
// Escape output to prevent XSS
echo '<p>Password was sent to: ' .
htmlspecialchars($submittedEmail, ENT_QUOTES, 'UTF-8') .
'</p>';
}
}
// User clicked confirmation link from email
} elseif (isset($_GET['cpw']) && isset($_GET['npw'])) {
$uid = (int)$_GET['npw'];
$cpw = preg_replace('#[^a-zA-Z0-9]#', '', $_GET['cpw']);
if (!$database->resetPassword($uid, $cpw)) {
echo '<p>The password has not been changed. Perhaps the activation code has already been used.</p>';
} else {
echo '<p>The password has been successfully changed.</p>';
}
// Display form
} else {
?> ?>
<p>Before you can request a new password you have to enter the email address that has been used to register the account.
<br /><br />Afterwards you will receive an e-mail with a new password. The password will only work after confirming it, though.</p>
<form action="password.php" method="post">
<p>
<b>Email</b><br />
<input type="hidden" name="npw" value="<?php echo intval($_GET['npw']); ?>" />
<input class="text" type="text" name="email" maxlength="50" />
</p>
<p> <p>
<button value="ok" name="s1" class="trav_buttons" id="btn_ok" alt="OK" /> ok </button> Before you can request a new password you have to enter the email
</p> address that has been used to register the account.
</form> <br /><br />
<?php Afterwards you will receive an e-mail with a new password.
} The password will only work after confirming it, though.
?> </p>
</div>
<div id="side_info" class="outgame"> <form action="password.php" method="post">
<p>
<b>Email</b><br />
<input type="hidden"
name="npw"
value="<?php echo (int)$_GET['npw']; ?>" />
<input class="text"
type="text"
name="email"
maxlength="50" />
</p>
<p>
<button value="ok"
name="s1"
class="trav_buttons"
id="btn_ok"
alt="OK">
ok
</button>
</p>
</form>
<?php } ?>
</div> </div>
<div id="side_info" class="outgame"></div>
<div class="clear"></div> <div class="clear"></div>
</div> </div>
<div class="footer-stopper outgame"></div> <div class="footer-stopper outgame"></div>
<div class="clear"></div> <div class="clear"></div>
<?php include("Templates/footer.tpl"); ?> <?php include("Templates/footer.tpl"); ?>
<div id="ce"></div> <div id="ce"></div>
</body> </body>
</html> </html>