这是python eval（）的安全使用吗？

Question

如果攻击者可以控制的值attacker_controlled_nasty_variable ，是这个代码段脆弱？

dic={"one":1,
      "nasty":attacker_controlled_nasty_variable,
     }
store=str(dict)
...
dic=eval(store)

Answer 1

使用ast.literal_eval()而不是eval() 。

Answer 2

是。 它可以替换为具有__repr__()方法的对象，该方法具有有效负载本身，或者返回传递给eval()时可能不安全的字符串。

概念证明：

class MyClass(object):
  def __repr__(self):
    return 'os.system("format c:")'

bad = [MyClass()]
print str(bad)

Answer 3

它是安全的，只要你可以确定attacker_controlled_nasty_variable永远不是攻击者可以控制__repr__ （或__str__ ）的对象，否则它会注入python代码。

但是，最好使用repr(dic)而不是str(dic)因为只有repr才能返回有效的python代码。

另外 - 正如@payne所提到的 - 使用更安全的ast.literal_eval()而不是eval() 。

Answer 4

我们来试试吧：

>>> attacker_controlled_nasty_variable="`cat /etc/passwd`"
>>> dic={"one":1,
...     "nasty":attacker_controlled_nasty_variable,
... }
>>> store = repr(dic)
>>> store
"{'nasty': '`cat /etc/passwd`', 'one': 1}"
>>> dic=eval(store)
>>> dic
{'nasty': '`cat /etc/passwd`', 'one': 1}

>>> attacker_controlled_nasty_variable="'hello',}"
>>> dic={"one":1,
...     "nasty":attacker_controlled_nasty_variable,
... }
>>> repr(dic)
'{\'nasty\': "\'hello\',}", \'one\': 1}'
>>> eval(repr(dic))
{'nasty': "'hello',}", 'one': 1}

您可能想尝试更多案例，但从经验__repr__正在引用内容。