这是我当前的代码:
<?php
$key = $_REQUEST['key'];
$url = $_REQUEST['url'];
include_once '../../dbconnect.php';
$query = $conn->query("SELECT * FROM members WHERE apikey='$key' && status='Active'");
$userRow=$query->fetch_array();
$conn->close();
/// Verify the URL starts with http:// or https://
if (0 === strpos($url, 'http://') || 0 === strpos($url, 'https://')) {
$url = $url;
} else {
$url = "http://$url";
}
/// Verify the key is 32 characters
if (!preg_match('/[^A-Za-z0-9]/', $key) && (strlen($key) == 32)) {
/// Verify the URL isn't malicious
if (filter_var($url, FILTER_VALIDATE_URL) === FALSE) {
die('Error: Invalid URL');
} else {
if ($userRow['status'] === 'Active') {
function generateRandomString($length = 8) {
$characters = '0123456789abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ';
$charactersLength = strlen($characters);
$randomString = '';
for ($i = 0; $i < $length; $i++) {
$randomString .= $characters[rand(0, $charactersLength - 1)];
}
return $randomString;
}
$redirect = generateRandomString();
$addshort = $conn->query("INSERT INTO short_".$redirect[0]." (redirect, apikey, url) VALUES ('".$redirect."','".$key."','".$url."')");
if ($conn->query($addshort) === TRUE) {
echo "added correctly";
} else {
echo "there was an error";
}
$conn->close();
} else {
echo "Error: Account Not Active";
}
}
} else {
die('Error: Invalid API Key');
}
?>
Run Code Online (Sandbox Code Playgroud)
这是错误日志:
[18-Oct-2016 12:21:31 America/New_York] PHP Warning: mysqli::query(): Couldn't fetch mysqli in /home/username/public_html/subdomains/url/index.php on line 40
[18-Oct-2016 12:21:31 America/New_York] PHP Warning: mysqli::query(): Empty query in /home/username/public_html/subdomains/url/index.php on line 42
[18-Oct-2016 12:21:31 America/New_York] PHP Warning: mysqli::close(): Couldn't fetch mysqli in /home/username/public_html/subdomains/url/index.php on line 48
Run Code Online (Sandbox Code Playgroud)
你可以在第 7 行看到我第一次连接数据库的地方:
$query = $conn->query("SELECT * FROM members WHERE apikey='$key' && status='Active'");
Run Code Online (Sandbox Code Playgroud)
那条线正在起作用。但是,我第二次连接做一个INSERT,我收到上述错误:
$addshort = $conn->query("INSERT INTO short_".$redirect[0]." (redirect, apikey, url) VALUES ('".$redirect."','".$key."','".$url."')");
Run Code Online (Sandbox Code Playgroud)
只是盯着这段代码太久了,我有什么遗漏吗?
您的连接已经关闭,在执行第一个 SELECT 语句后,这意味着连接提前关闭:
$conn->close();
Run Code Online (Sandbox Code Playgroud)
您需要close()在所有查询之后使用或重新建立连接。第一个是更好的选择。
您正在获取用户输入$_REQUEST['key'],这意味着您的查询已针对 SQL 注入开放,这将帮助您了解如何使用 SQL 注入防止代码:如何在 PHP 中防止 SQL 注入?