我想知道在這段代碼中是否有錯誤/漏洞利用,也有人可以幫助我,因爲我註冊了它,但它並沒有將數據插入到數據庫中。如果有任何錯誤,請您糾正。我想要它,所以如果用戶名存在,將它們重定向到錯誤?= 1,以此類推,密碼不匹配。任何幫助表示讚賞。PHP註冊問題
Register.php
<form action="register_acc.php" method="post">
<input type="text" name="username" class="input" value="" autocomplete="off" placeholder="Username" maxlength="25" /><br />
<br />
<input type="password" name="password" class="input" value="" autocomplete="off" placeholder="Password" maxlength="20" /><br />
<br />
<input type="password" name="password2" class="input" value="" autocomplete="off" placeholder="Password again" maxlength="20" /><br />
<br />
<input type="text" name="email" class="input" value="" autocomplete="off" placeholder="Email" maxlength="255" /><br />
<br />
<input type="submit" name="submit "class="submit" value="Sign up">
</form>
register_acc.php
<?php
error_reporting(1);
include 'site/inc/config.php';
if (isset($_POST['submit'])) {
session_start();
$username = $_POST['username'];
$password = md5($_POST['password']);
$pass_conf = md5($_POST['password2']);
$email = $_POST['email'];
$ip = $_SERVER['REMOTE_ADDR'];
$date= date("d-m-Y");
$q = "SELECT * FROM `users` WHERE username = '$username'";
$r = mysql_query($q);
if (empty($username)) {
header("Location: register.php?error=1");
exit;
}
if ($password != $pass_conf) {
header("Location: /site/register.php?error=2");
exit;
}
if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
header("Location: /site/register.php?error=3");
exit;
}
if (mysql_num_rows($r) == 0) {
// Continue w/ registration, username is available!
$query = "INSERT INTO `users` (id, username, password, email, ip, rank, reg_date)
VALUES (0, '$username', '$password', '$email', '$ip', 1, '$date'())";
$run = mysql_query($query);
header("Location: /site/register.php?succsess=1");
}
}
else {
header("Location: register.php?error=4");
}
?>
爲什麼你插入這樣的:$查詢=「INSERT INTO'users'(ID,...) VALUES(0,... )「; 如果設置在數據庫中的auto_increment 此外,喲有另一個錯誤INT可以ommit插入他的ID插入:VALUES(...,「$日期」()),你不需要「()」後「 $ date' – 2013-04-20 11:55:41
你的腳本容易受到[SQL注入](http://php.net/manual/en/security.database.sql-injection.php),'mysql_ *'函數也被棄用,並且在新代碼中應該使用'PDO'或'MySQLi'代替 – kero 2013-04-20 12:04:40
感謝您的幫助,更新它 @Kingkero你能修補漏洞或者告訴我他們在哪裏,而且我會很快切換到庫MySQLi。由於 – 2013-04-20 12:06:58