首先,这个漏洞属于网站coding写法不规范,官方给了提示:
但官方还是做了修补,升级最新版本V5.8.7可修复。
我们先定位下这里:

Illuminate\Validation\Rule
官方推举的写法是:
Rule::unique('users')->ignore($id),
如果网站coding没有预先对$id的值做处理时,用户可以直接通报恶意数据给ignore函数,就会导致SQL注入。
我们来跟一下函数:
\Illuminate\Validation\Rules\Unique.phpclass Unique{...public function ignore($id, $idColumn = null) { if ($id instanceof Model) { return $this->ignoreModel($id, $idColumn); } $this->ignore = $id; $this->idColumn = $idColumn ?? 'id'; return $this; }
这里我们不考虑把$id写成实例的情形,$id是用户可控的话,$idColumn直接写为空即可,末了赋值情形如下:
$this->ignore = $id;$this->idColumn = 'id';
如果网站代码类似这样布局的话,黑客输入的值就属于可控状态:
$id = $request->input('id');
末了我们会走到这儿:
Illuminate\Validation\Rules\Unique.php public function __toString() { ... ... }
我们看下关键的代码变更:
Illuminate\Validation\Rules\Unique.phpV5.8.7【最新版】 public function __toString() { $this->ignore ? '\"大众'.addslashes($this->ignore).'\"大众' : 'NULL', }Illuminate\Validation\Rules\Unique.phpV5.8.4 public function __toString() { $this->ignore ? '\公众'.$this->ignore.'\公众' : 'NULL', }
这里最新的代码v5.8.7,把$this->ignore直接给addslashes了,以前这里是没有防护的。
有趣的是,笔者比拟了下diff,期间官方还试图对其他引用的地方进行过滤。末了还是在__toString处,进行了统一的过滤。
末了提一句,后面的代码会进入DatabaseRule,进行后续SQL规则匹配。
Illuminate\Validation\Rules\DatabaseRule.php
这之后就没有再进一步处理,接着形成了SQL注入。
阅读链接如下:
官方通知布告
解释文档
本文作者:dawner,转载自FreeBuf.COM