这行代码有什么意义呢？

Question

我在administrator/components/com_virtuemart/classes/ps_product.php的第2136行的Joomla Virtuemart插件中找到了这行代码

eval ("\$text_including_tax = \"$text_including_tax\";");

Answer 1

废弃我之前的答案。

这个 eval() 出现在这里的原因在 php eval 文档 中显示，

这就是发生的事情

$text_including_tax = '$tax <a href="...">...</a>';

...

$tax = 10;

...

eval ("\$text_including_tax = \"$text_including_tax\";");

：此 $text_include_tax 的末尾等于：

"10 <a href="...">...</a>"

单引号可防止将 $tax 包含在字符串的原始定义中。 通过使用eval()，它会强制它重新计算字符串并将$tax 的值包含在字符串中。

我不喜欢这种特殊方法，但它是正确的。 另一种方法是使用 sprintf()

Answer 2

这段代码似乎是强制 $text_include_tax 成为字符串的糟糕方法。

它不好的原因是因为 if $text_include_tax 可以包含用户输入的数据，他们就有可能执行任意代码。

例如，如果 $text_include_tax 设置为等于：

"\"; readfile('/etc/passwd'); $_dummy = \"";

eval 将变为：

eval("$text_include_tax = \"\"; readfile('/etc/passwd'); $_dummy =\"\";");

向恶意用户提供 passwd 文件的转储。

更正确的方法是将变量转换为 string:

$text_include_tax = (string) $text_include_tax;

甚至只是：

$text_include_tax = "$text_include_tax";

如果数据 $text_include_tax 只是内部变量或包含已验证的内容，则不存在安全风险。 但这仍然是一种将变量转换为字符串的糟糕方法，因为有更明显、更安全的方法可以做到这一点。

Answer 3

我猜测这是强制 $text_include_tax 成为字符串而不是数字的一种时髦方式。

Answer 4

也许这是尝试将变量转换为字符串？ 只是一个猜测。

Answer 5

您将需要评估才能将税率输入到输出中。 刚刚将其移至新服务器，由于某种原因，该行导致服务器错误。 作为快速修复，我将其更改为：

//eval ("\$text_including_tax = \"$text_including_tax\";");
$text_including_tax = str_replace('$tax', $tax, $text_including_tax);

Answer 6

它将字符串评估为 PHP 代码。

但它似乎让一个变量等于它自己？ 诡异的。

Answer 7

正如其他人指出的那样，这是由不知道自己在做什么的人编写的代码。

我还快速浏览了代码，发现在放置 HTML/URI 等时完全没有文本转义。 一起。 如果您愿意正确审核的话，除了评估问题之外，这里可能还可以找到许多注入漏洞。

我不希望这段代码在我的服务器上运行。

Answer 8

我之前已经浏览过该代码库。 这是我见过的最糟糕的 PHP 之一。

我想你会做这种事情来掩盖你在其他地方犯的错误。

Answer 9

不，它是这样做的：

说 $text_include_tax = "flat"。 此代码计算以下行：

$flat = "flat";

这不一定很好，但我确实使用过这样的技术一次将所有 MySQL 变量吸收到一个数组中，如下所示：

    while ($row = mysql_fetch_assoc($result)) {
        $var = $row["Variable_name"];
        $var = $row["Value"];
    }