网站被黑的用户名、附加信息和密码已更改问题？

Question

我想知道什么可以让用户破解我的网站，他们更改了我的用户名、个人信息和密码。有人可以给我一些关于它可能是什么的建议吗？我正在使用 PHP MySQL 和 HTMLPURIFIER。

这是登录脚本。

<?php
if (isset($_POST['submitted'])) { // start of submit conditional.
    require_once (MYSQL);

    // Validate the username or email address:
    if (!empty($_POST['login']) && strlen($_POST['login']) <= 255) {
        $e = mysqli_real_escape_string($dbc, $purifier->purify(strip_tags($_POST['login'])));
    } else if(!empty($_POST['login']) && strlen($_POST['login']) >= 256) {
        $e = FALSE;
        echo '<p>Your username or email address cannot exceed 255 characters!</p>';
    } else {    
        $e = FALSE;
        echo '<p>You forgot to enter your username or email address!</p>';
    }

    // Validate the password:
    if (!empty($_POST['pass']) && strlen($_POST['pass']) <= 255) {
        $p = mysqli_real_escape_string($dbc, $_POST['pass']);
    } else if(!empty($_POST['pass']) && strlen($_POST['pass']) >= 256) {
        $p = FALSE;
        echo '<p>Your password cannot exceed 255 characters!</p>';
    } else {
        $p = FALSE;
        echo '<p>You forgot to enter your password!</p>';
    }

    if(($e != FALSE) && ($p != FALSE)) { // check pass
        $pass_salt = "SELECT users.password, users.salt FROM users JOIN contact_info ON contact_info.user_id = users.user_id WHERE (contact_info.email = '" . $e . "' OR users.username = '" . $e . "') AND users.active IS NULL";
        $ph = mysqli_query($dbc, $pass_salt) or trigger_error("Query: $pass_salt\n<br />MySQL Error: " . mysqli_error($dbc));

        while($row = mysqli_fetch_array($ph)){ 
            $password = $row['password'];
            $salt = $row['salt'];
        }

        if(!empty($salt)) {
            $sha512 = hash('sha512', $p . $salt);
        }

        if(!empty($password) == !empty($sha512)){
            $user_pass = TRUE;
        } else {
            $user_pass = FALSE;
        }
    }


    if(isset($user_pass) && ($user_pass == TRUE) && !empty($salt)) { // If everything's OK.
        // Query the database:
        $q = "SELECT users.user_id, users.first_name, users.user_level FROM users JOIN contact_info ON contact_info.user_id = users.user_id WHERE (contact_info.email = '" . $e . "' OR users.username = '" . $e . "') AND users.password = '" . $sha512 . "' AND users.active IS NULL";        
        $r = mysqli_query ($dbc, $q) or trigger_error("Query: $q\n<br />MySQL Error: " . mysqli_error($dbc));


        if (@mysqli_num_rows($r) == 1) { // A match was made.

            // Register the values & redirect:
            $_SESSION = mysqli_fetch_array ($r, MYSQLI_ASSOC); 
            // check if user is logged in then update the old login date
            $u = "UPDATE users JOIN contact_info ON contact_info.user_id = users.user_id SET users.last_login = NOW(), users.deletion = 0, users.deletion_date = NULL WHERE (contact_info.email = '" . $e . "' OR users.username = '" . $e . "') AND users.password = '" . $sha512 . "' AND users.active IS NULL"; 
            // save the info to the database
            $r = mysqli_query ($dbc, $u);
            mysqli_free_result($r);
            mysqli_close($dbc);

            $url = BASE_URL . 'home/index.php'; // Define the URL:
            header("Location: $url");
            exit(); // Quit the script.

        } else { // No match was made.
            echo '<p>Either your username, email address or password entered do not match those on file or you have not yet activated your account.</p>';
        }

    } else { // If everything wasn't OK.
        echo '<p>Please try again.</p>';
    }

    mysqli_close($dbc);

} // end of submit conditional.
?>

Answer 1

您应该了解 SQL 注入。这就是我首先想到的（注意到 MySql 的使用）。为了防止这种情况，您必须使用mysql_real_escape_string()（不同的mysql_escape_string()，它被视为已弃用）来清理用户输入。尽管有这个解决方案，我建议您使用 PDO 或 Mysqli （我通常不鼓励使用这个），以便通过使用来修复 SQL 注入问题准备好的语句。

那么您可能应该意识到 XSS（跨站点脚本）可能会在您的代码中“注入”某种恶意 Javascript 脚本。您可以使用 htmlspecialchars() 稍微修复此问题，使 HTML 标记（例如

另请查看PHP 漏洞列表。

PS

为了使您的代码更具可读性和“正确”，我建议您将 strlen($_POST['login']) >= 256 更改为 <代码>strlen($_POST['login'])>; 255 是相同的，但让读者立即明白真正的限制不是 256 而是 255。

Answer 2

首先，确保他们无法执行 SQL 注入。这可能就是他们所做的。这通常是由执行的输入字段引起的。执行此操作的人只需输入 SQL 命令即可。

您可以在此处查看详细信息。

哦，欢迎来到 StackOverflow.com！我希望您喜欢这个网站！

Answer 3

除了 DalexL 答案之外，请检查您是否有用于连接数据库的强密码。

Answer 4

我很确定这没有发生在原始发帖者身上，但众所周知，欺骗性公司会向拥有网站的公司发送“账单”。 “账单”中的细则将域名从该公司转移到了欺骗性公司。

以下是一个示例：美国域名注册局骗局

Answer 5

如果您允许用户上传图像，您可能已成为 GIF 漏洞的受害者。如果您的服务器设置不安全，则查看嵌入 PHP 代码的 GIF 将会执行该代码。检查您的系统上是否可以找到任何 .gif.php（或 .php.gif）文件，如果黑客忘记自行清理，它们可能仍然存在。

Answer 6

如果 HTMLPURIFIER 对于所有面向数据库的输入正确实现，那么您应该评估您如何传输登录信息。您在客户端提交之前是否进行了预哈希处理？

我假设原因是表单上有一个未经过滤的输入，并且它让一些 SQL 注入代码通过。