如何在 PHP 代码中的 file_get_contents 前面添加域

Question

我是新来的，有一个 PHP 编码器，可以对我的 2 个文件进行一些 mods p>

这是已编辑的代码（您可以使用“DrTech76”查看已编辑的部分）-:

get_image.php

<?php
require_once 'include/db.php';
$file_name = $_GET['f'];
if ( empty( $file_name ) || !file_exists( 'uploads/' . $file_name ))
{
    die( 'File not found...' );
}

$sql = "SELECT original_file_name FROM nfo_images WHERE file_name = '" . $file_name . "'";
$rs = mysql_query( $sql );
$file = mysql_fetch_assoc( $rs );

$original_file_name = explode( '.', $file['original_file_name'] );
//just get rid of the last, then put back together
array_pop( $original_file_name );
$original_file_name = implode( '.', $original_file_name );

header("Content-Type: image/png");
//start edit 10.07.2014 DrTech76, flow the direct image link through here to keep the actuall location unknown
if(!isset($_REQUEST["dl"]))
{//it's not a request through a direct link, so process as file download, as it was   originally
    header("Content-Disposition: attachment; filename=\"" . $original_file_name . ".png\"");
}
//end edit 10.07.2014 DrTech76, flow the direct image link through here to keep the actuall location unknown
echo file_get_contents( 'uploads/' . $file_name );
?>

另一个编辑的文件是 index.php（添加的输入区域，应该是文本区域 IMO，带有“选择复制”以将内容复制到剪贴板）（这只是我添加的已编辑区域，否则会太长）

<div id="uploaded_image_div" align="center">
            <b>Left Click on the Below Image to Save it to your PC...</b><br /><br />
            <div id="title" style="font-weight:bold;"></div>
            <br />
            <a href="" target="_blank"><img border="0" src="" /></a>
            <br />
            <?php
            //start edit 10.07.2014 DrTech76, direct link containers, styling is done  through css/style.css
            ?>
            <div id="direct_link">
                <label for="link">Direct link to this image</label><br />
                <input id="link" readonly onfocus="$(this).select()" />
            </div>
            <?php
            //end edit 10.07.2014 DrTech76, direct link containers, styling is done through css/style.css
            ?>


            <br />
            <br />

有人可以帮我解决这个错误吗，正如您从 NFOPic.com 索引页面中看到的那样，实际的域名没有显示 - 虽然我需要它，所以如果我将它上传到另一个域（一个新的测试版发布时选择的域）它将适用于任何域，而不仅仅是将 http://nfopic.com/ 放在代码前面 - 提前致谢！

Answer 1

所以，这个脚本有很多事情要做。

1）摆脱XSS和SQL注入漏洞：

现在，您正在检查文件是否存在，这并非完全错误，但不能为您完成这项工作：

$file_name = $_GET['f'];
if ( empty( $file_name ) || !file_exists( 'uploads/' . $file_name ))
{
  die( 'File not found...' );
}

既然彼得已经告诉过你，你允许人们通过在浏览器中插入http://nfopic.com/get_image.php?f=../include/db.php 来下载你的 SQL Server 访问数据，我假设你理解了基本问题。 file_exists 只是检查文件在服务器上是否可用，这对于您的 include/db.php 文件是正确的。因此解释器跳过 IF 并继续执行。由于您已经在使用数据库，我们将使用它来确定文件是否真的存在。我们将在几秒钟内完成。

mysql_query 根本不安全（SQL 注入）！您必须使用 PDO 或 MySQLi 进行 SQL 查询（我使用 PDO 进行演示）：

$prep = $db->prepare("SELECT original_file_name FROM nfo_images WHERE file_name = ?");
$prep->execute(array($file_name));
$res = $prep->fetchAll();
if(count($res) == 0){
  die( 'File not found...' );
}
$original_file_name = explode('.', $res[0]["original_file_name"]);
// ... and what else you want to do with the file name

您可以在此处阅读更多信息：http://wiki.hashphp.org/PDO_Tutorial_for_MySQL_Developers

现在我们应该已经通过这种方式关闭了这两个漏洞（假设上传不允许与数据库进行错误交互->您需要检查generate_image.php）。

2) 您在添加域时遇到的问题：

我不太明白你的问题是什么，所以如果我没有正确回答，请随时发表评论并提出更多问题。

-您可以通过调用$_SERVER['SERVER_NAME'] 获取当前域。这应该输出类似：nfopic.com（不过我不知道你想在哪里添加）。

- 如果您的意思是input 字段中的域名错误，您可以在js/nfo.js 文件中更改它。有一行：

$('#uploaded_image_div #link').val('http://nfopic.com/get_image.php?f=' + validate[1]);

http://nfopic.com/get_image.php?f= 部分是硬编码的，因此您必须以任何其他方式插入它。

- 如果您打算将域名添加到用户可以下载的图像名称中，您必须使用.htaccess 文件重定向您的图像，以便您可以请求真实图像姓名image_234234.png。

3) 主页的其余部分：

由于网站这部分有很多漏洞和错误，generate_image.php中肯定有一些。文件类型（txt、nfo 和 diz）的检查在 JS 中完成。问题在于，如果 PHP 中也没有一些检查，您可以解决它们并上传任何其他文件类型。这导致我们发现另一个漏洞 -> 可以上传一个 SHELL 脚本（简单的 PHP 文件），并且您的文件可能会被攻击者修改。

终于： 整个主页可能不安全。肯定有一些修改需要做。 再说一次，如果我没有回答您的开始问题，请随时提出更多问题。