我有一个非常奇怪的bug,我根本不理解。我有下面的PHP脚本,它有两种行为方式。
这个脚本shell将新用户添加到mysql数据库中。我想做的是检查数据库中是否已经有用户名。如果是,请设置一个用于决定如何继续的变量。如果我执行这个代码,这行
if(strcmp($row["username"], $addUser_name) == 0)
执行(或者更好地跳进去),但我100%确信,在执行此行之前,数据库中不存在(或者不应该存在)该名称。现在,即使$errorName设置为1并且正在打印测试,我想添加到数据库中的数据集也出现在我的数据库中,但最后一部分从开始
if(!isset($errorName))
不执行。我用简单的回声语句一遍又一遍地检查它。
为了查看我的数据集,我删除了中的注释
while ($row = mysqli_fetch_assoc($allUserNames))
{
echo $row["username"]."'n";
}
该部分正在正确执行
if(!isset($errorName))
但我在数据库中找到了两次数据集。
我完全不明白为什么科学院会这样做。我尝试了很多不同的事情,但我不明白,我做错了什么。
<?php
include "auth/auth1.php";
include "functions/connectToDB.php";
include "functions/test_input_XSS.php";
if(isset($_GET["startCheck"])) //TODO Mache auch GET noch POST
{
//Sollte niemals true sein! Passiert nur, wenn man Unsinn macht
if(strcmp($_GET["addUser_pw"], $_GET["addUser_pwRepeat"]) !== 0) { die; }
$servername = "localhost";
$databasename = "X";
$mysqluser = "X";
$mysqlpass = "X";
$addUser_name = $_GET["addUser_name"];
$connection = connectToDB($servername, $mysqluser, $mysqlpass, $databasename);
if(mysqli_connect_errno())
{
printf("Connect failed!");
die();
}
$query_getAllUserNames = "SELECT username FROM user;";
$allUserNames = mysqli_query($connection, $query_getAllUserNames);
/*while ($row = mysqli_fetch_assoc($allUserNames))
{
echo $row["username"]."'n";
}*/
while ($row = mysqli_fetch_assoc($allUserNames))
{
if(strcmp($row["username"], $addUser_name) == 0)
{
$errorName = 1;
echo "test";
}
}
if(!isset($errorName))
{
$username = test_input_for_XSS($_GET["addUser_name"]);
$password = hash("sha256", $_GET["addUser_pw"]);
$permission = test_input_for_XSS($_GET["addUser_permission"]);
$query_addUser = "INSERT INTO user (username, passwordhash, permissionlevel) VALUES ('".$username."', '".$password."', '".$permission."');";
$addUserSuccess = mysqli_query($connection, $query_addUser);
if($addUserSuccess !== 1)
{
$hostname = $_SERVER['HTTP_HOST'];
$path = dirname($_SERVER['PHP_SELF']);
echo"Success";
//header("Location: http://".$hostname.($path == "/" ? "" : $path)."/userManagment.php?added=".$username."");
}
else
{
echo "ANNOYING_ERROR";
}
}
//Tidy up
mysqli_free_result($allUserNames);
mysqli_close($connection);
}
?>
这是相应的HTML代码,紧接着在同一个文件中:
<?php
include "home.php";
?>
<section>
<h3>Einen neuen Benutzer hinzufügen</h3>
<?php
if(isset($errorName))
{
echo '<p class="warningMessage">Der Nutzername <b>'.$_GET["addUser_name"].'</b> ist bereits vergeben.<br />Bitte wählen Sie einen anderen aus!</p>';
}
?>
<form method="GET" action="addUser.php">
<table>
<tr>
<td>Nutzername:</td>
<td><input type="text" name="addUser_name" required pattern="'w+" /></td>
<td></td>
<td class="annotation">z.B.: Vorname</td>
</tr>
<tr>
<td>Passwort:</td>
<td><input type="password" name="addUser_pw" required pattern="(?=.*'d)(?=.*[a-z])(?=.*[A-Z]).{6,}" title="Passwort muss mindestens 6 Zeichen, Groß- und Kleinbuchstaben, sowie Zahlen enthalten"
onChange="this.setCustomValidity(this.validity.patternMismatch ? '' : ''); if(this.checkValidity()){ form.addUser_pwRepeat.pattern = this.value; }" /></td>
<td></td>
<td class="annotation">Muss Groß- und Kleinbuchstaben, Zahlen und mindestens 6 Zeichen enthalten</td>
</tr>
<tr>
<td>Passwort wiederholen:</td>
<td><input type="password" name="addUser_pwRepeat" required pattern="(?=.*'d)(?=.*[a-z])(?=.*[A-Z]).{6,}" /></td>
<td></td>
<td class="annotation">Muss identisch sein zum ersten eingegebenen Passwort</td>
</tr>
<tr>
<td>Berechtigungslevel:</td>
<td>
<input type="radio" name="addUser_permission" value="1" checked />1<br />
<input type="radio" name="addUser_permission" value="2" />2
</td>
</tr>
</table>
<input type="hidden" name="startCheck" value="1" />
<input type="submit" value="Nutzer hinzufügen" class="button" />
</form>
</section>
</body>
</html>
如果我的问题描述得不够清楚,我很乐意提供所需的任何其他信息。
我建议不要通过PHP获取所有用户名并检查是否存在相同的用户名。你可以简单地进行这样的查询
$result=$connection->query("SELECT username FROM user WHERE username='".$connection->real_escape_string($addUser_name)."';");
然后检查查询是否返回任何行
if($result -> num_rows > 0)
{
//the username is already in use
}
else
{
//the username is unique
}
您正在检查数据库中的所有用户名,并在while循环中循环它们。
因此,在每次迭代中,当您在while范围变量的值为结果集中的最新用户设置时使用$errorName
变量时,您将为每个用户名覆盖$errorName
变量。
你能做的就是
$query_getAllUserNames = "SELECT count(*) FROM user where username=?";
if ($stmt = mysqli_prepare($connection, $query_getAllUserNames)) {
/* bind parameters for markers */
mysqli_stmt_bind_param($stmt, "s", $addUser_name);
/* execute query */
mysqli_stmt_execute($stmt);
/* bind result variables */
mysqli_stmt_bind_result($stmt, $userCount);
/* fetch value */
mysqli_stmt_fetch($stmt);
}
if(!isset($userCount) || $userCount == 0) {
//Create a new user here
echo "Create new user";
} else {
//Go with the flow
echo "User exists";
}
//rest of your code