你能发现SQL注入吗？

Question

我读了2600年的文章，但本文也在这里，https://viaforensics.com/mobile-security/static-code-analysis-watchtower.html --无论如何，有一个代码块：

$result = mysql_query("SELECT * FROM users WHERE username = '{$_GET['username']}' AND `password` = SHA1('{$_GET['password']}')")

作者说：“2600的读者会发现明显的SQL注入，但似乎许多程序员--特别是--不会。”有人能解释并指出他的意思吗？

对我来说，我猜他的意思是，既然似乎没有清理那些因注入而声名狼藉的字符的数据，那么它是脆弱的吗？

我对PHP5 5/MySQL还比较陌生，我反复研究了这段代码，找出了哪里出了问题，但没有得出任何其他结论。

Answer 1

如果username类似于'' OR ''=''#，而password是'anything'，那么查询就会短路：

SELECT *
FROM users
WHERE username ='' OR ''=''#AND password ='anything' 

您可以通过将SQL注入参数来短路逻辑。

Answer 2

减轻SQL注入的最佳方法是在所有SQL查询中使用参数。

如果连接字符串以创建查询，则可以降低注入的风险，但这不是最好的方法。你可以假设，如果你连在一起，总是有风险的。在上述查询的情况下，您可以将任何想要的值注入参数。

Answer 3

让我们扩展一下两栖动物的答案，$_GET‘参数’是URL的一部分，所以它看起来很像。

http://link/foo.php?username=thatguy&password=whoa

其中$_GET‘’username‘是"whoa“，$_GET’‘password’是”哇“。

让我们把它写进代码中。

$result = mysql_query("SELECT * FROM users WHERE username = 'thatguy' AND `password` = SHA1('whoa')")

如果我们传入用户名的"' OR ''=''#“会发生什么，让我们将其编码为URL。

http://link/foo.php?username=%27%20OR%20%27%27%3D%27%27%23&password=whoa

这个会回来的

$result = mysql_query("SELECT * FROM users WHERE username = '' OR ''=''# AND `password` = SHA1('whoa')")

' OR ''=''#将强制查询返回所有结果，而#将注释掉MySQL语句的其余部分，因此谁关心密码。

因此，MySQL查询将只查看

SELECT * FROM users WHERE username = '' OR ''=''

只需要有人来确认URL编码。