15

我在 php 中有一个网站,它确实 include() 将内容嵌入到模板中。要加载的页面在 get 参数中给出,我将“.php”添加到参数的末尾并包含该页面。我需要做一些安全检查以避免 XSS 或其他东西(不是 mysql 注入,因为我们没有数据库)。我想出的是以下内容。

$page = $_GET['page'];

if(!strpos(strtolower($page), 'http') || !strpos($page, '/') ||
    !strpos($page, '\\') || !strpos($page, '..')) {
        //append ".php" to $page and include the page

我还能做些什么来进一步清理我的输入吗?

4

3 回答 3

34
$page = preg_replace('/[^-a-zA-Z0-9_]/', '', $_GET['page']);

可能是清理它的最快方法,这将采取任何措施并确保它只包含字母、数字、下划线或破折号。

于 2009-10-19T09:24:43.147 回答
7

不要“清理”——攻击特定于数据的使用,而不是源头。在输出值时转义值。另请参阅我对使用 PHP 清理用户输入的最佳方法是什么的回答?

于 2009-10-19T10:23:04.390 回答
5

在源代码中定义一个明确的页面列表,然后使用它来检查输入。是的,这是更多的工作,但它非常清楚什么是允许的,什么是不允许的。例如:

$AVAILABLE_PAGES = array('home', 'news',  ...);
$AVAILABLE_PAGES = array_fill_keys($AVAILABLE_PAGES, 1);

$page = $_GET['page'];
if (!$AVAILABLE_PAGES[$page]) {
   header("HTTP/1.0 404 Not Found");
   die('Page not found.');
}

include "pages/$page.php";
于 2009-10-19T09:31:26.080 回答