0

所以基本上,我正在尝试制作一个简单但安全的忘记密码脚本。

有两个脚本,一个允许用户输入他们的电子邮件地址。然后,这将向他们发送一封电子邮件,其中包含他们必须访问以保存新密码的链接。

第二个脚本是链接指向的地方。此脚本保存新密码。

出于安全考虑,我在我的数据库中创建了一个名为“token”的新表。它具有三个字段;令牌,电子邮件,使用。Token 是随机生成的 10 个字母和数字组成的字符串,email 就是用户的邮箱地址,used 是 1 或 0 的整数,表示 token 是否被使用过。

阅读完这两个脚本后,您将能够更多地理解我的结构。它们不长,也不复杂。

出了什么问题

好的,所以只有一件小事出了问题,这在 reset-password.php 脚本中。这是用户在收到电子邮件后来到的地方。基本上,我输入一个新密码,然后单击“重置密码”,但没有任何反应。没有显示错误或确认,我的数据库中也没有任何变化。我似乎无法调试这个,并且已经搜索和尝试了几个小时。所有帮助和建议将不胜感激。

请记住,我还是 PHP 和 MySQL 的新手。已经使用 PHP 大约 8 周了,而 MySQL 仅使用了 2 周。

忘记密码.php

<?php
//Forgotten password script

    //Variable to save errors
    $errors = array();

    $email = $_POST['email'];

    include 'config.php';
    mysql_connect("$db_host", "$db_username", "$db_password")or die("cannot connect"); 
    mysql_select_db("$db_name")or die("cannot select DB");

    $query = "SELECT email FROM users WHERE email='" . $email . "'";
    $result = mysql_query($query);
    $num = mysql_num_rows($result);
    if($num==0)
    {
        echo ("<div style='color:red;'>Email address is not registered</div>");
        die();
    }

    $token = getRandomString(10);
    $query = "INSERT INTO tokens (token,email) VALUES ('".$token."','".$email."')";
    mysql_query($query);

    //function to renerate the token
    function getRandomString($length) 
    {
        $validCharacters = "ABCDEFGHIJKLMNPQRSTUXYVWZ123456789";
        $validCharNumber = strlen($validCharacters);
        $result = "";

        for ($i = 0; $i < $length; $i++) 
        {
            $index = mt_rand(0, $validCharNumber - 1);
            $result .= $validCharacters[$index];
        }
        return $result;
    }

    //Send the reset link to the user
    function mailresetlink($to,$token)
    {
        $subject = "Password Reset";
        $message = '
        <html>
        <head>
        <title>Password Reset</title>
        </head>
        <body>
        <p>Click on the given link to reset your password <a href="http://domain.com/reset-password.php?token='.$token.'">Reset Password</a></p>

        </body>
        </html>
        ';
        $headers = "MIME-Version: 1.0" . "\r\n";
        $headers .= "Content-type:text/html;charset=iso-8859-1" . "\r\n";
        $headers .= 'From: Password Reset <noreply@domain.com>' . "\r\n";

        if(mail($to,$subject,$message,$headers))
        {
            echo "We have sent the password reset link to your email at <strong>".$to."</strong>"; 
        }
    }

    //If email is posted, send the email
    if(isset($_POST['email']))
    {
        mailresetlink($email,$token);
    }


?>




<table align="center" style="padding-bottom:40px;">
    <form action="<?php $_SERVER['PHP_SELF']; ?>" method="post">
    <tr>
    <td>Email Address: </td>
    <td><input type="text" name="email" /></td>
    </tr>
    <tr>
    <td colspan="2" align="center"><input type="submit" value="Reset My Password" /></td></tr>
    <input type="hidden" name="register" value="TRUE" />
</form>
</table>

重置密码.php

<?php
//Reset password script

    $token = $_GET['token'];
    $email;

    include 'config.php';
    mysql_connect("$db_host", "$db_username", "$db_password") or die("cannot connect"); 
    mysql_select_db("$db_name")or die("cannot select DB");

    if(!isset($_POST['newpassword']))
    {
        $query = "SELECT email FROM tokens WHERE token='" . $token . "' AND used = 0";
        $result = mysql_query($query);
        while($row = mysql_fetch_array($result))
        {
            $email = $row['email'];
        }


        if ($email != '')
        {
            $_SESSION['email'] = $email;
        }
        else 
        {
            echo "Invalid link or Password already changed";
        }
    }


    $pass = $_POST['newpassword'];
    $email = $_SESSION['email'];

    //Save new password
    if(isset($_POST['newpassword']) && isset($_SESSION['email']))
    {
        $query = "UPDATE users SET password = SHA('$password') WHERE email='" . $email . "'";
        $result = mysql_query($query);
        if($result)
        {
            mysql_query("UPDATE tokens SET used=1 WHERE token='" . $token . "'");
        }
        echo "Your password has been changed successfully";
        if(!$result)
        {
            echo "An error occurred. Please try the again or contact us at admin@domain.com";
        }
    }

?>



<table align="center" style="padding-bottom:40px;">
    <form action="<?php $_SERVER['PHP_SELF']; ?>" method="post">
    <tr>
    <td>New Password:</td>
    <td><input type="password" name="newpassword" id="password"/></td>
    </tr>
    <tr>
    <td colspan="2" align="center"><input type="submit" value="Change Password"></td></tr>
    <input type="hidden" name="reset" value="TRUE" />
</form>
</table>

请,如果您需要更多信息或代码,请随时询问。

提前致谢!

4

1 回答 1

5

输入新密码参数后,我看不到您在重置页面上将令牌参数传递给服务器的任何地方。<input />我希望 你应该有另一个隐藏的控制。$_SERVER['PHP_SELF']不返回查询字符串参数。这可能是您当前问题的原因。

具体来说,

<table align="center" style="padding-bottom:40px;">
    <form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
    <tr>
    <td>New Password:</td>
    <td><input type="password" name="newpassword" id="password"/></td>
    </tr>
    <tr>
    <td colspan="2" align="center"><input type="submit" value="Change Password"></td></tr>
    <input type="hidden" name="reset" value="TRUE" />
</form>
</table>

应该

<table align="center" style="padding-bottom:40px;">
    <form action="<?php echo $_SERVER['PHP_SELF']; ?>" method="post">
    <tr>
    <td>New Password:</td>
    <td><input type="password" name="newpassword" id="password"/></td>
    </tr>
    <tr>
    <td colspan="2" align="center"><input type="submit" value="Change Password"></td></tr>
    <input type="hidden" name="reset" value="TRUE" />
    <input type="hidden" name="token" value="<?php echo $_REQUEST['token']; ?>" />
</form>
</table>

确保您还将任何$_GET['token']s更改$_REQUEST['token']为,因为它将第一次是 GET,然后是第二次。

话虽如此,您更大的问题是我能够通过指定' or 1=1 or '为我的令牌来绕过您的所有安全性。或者,我可能是刻薄的,并且做得很好'; update users set password = SHA('IKnowThisPassword') where username = 'admin'; --

故事的寓意是参数化 SQL(如何防止 PHP 中的 SQL 注入?

于 2013-08-20T04:29:37.367 回答