PHP知名開發框架Laravel，之前在官方博客通報了一個高危SQL注入漏洞，這里簡單分析下。
首先，這個漏洞屬於網站coding寫法不規范，官方給了提示：
但官方還是做了修補，升級最新版本V5.8.7可修復。
我們先定位下這里：
Illuminate\Validation\Rule
官方推薦的寫法是：
Rule::unique('users')->ignore($id),
如果網站coding沒有預先對$id的值做處理時，用戶可以直接傳遞惡意數據給ignore函數，就會導致SQL注入。
我們來跟一下函數：
\Illuminate\Validation\Rules\Unique.php class 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';
鏈接：https://pan.baidu.com/s/1v5gm7n0L7TGyejCmQrMh2g 提取碼：x2p5
免費分享，但是X度限制嚴重，如若鏈接失效點擊鏈接或搜索加群 群號518475424。
如果網站代碼類似這樣構造的話，黑客輸入的值就屬於可控狀態：
$id = $request->input('id');
最后我們會走到這兒：
Illuminate\Validation\Rules\Unique.php public function __toString() {
...
...
}
我們看下關鍵的代碼變更：
Illuminate\Validation\Rules\Unique.php
V5.8.7【最新版】 public function __toString() { $this->ignore ? '"'.addslashes($this->ignore).'"' : 'NULL',
}
Illuminate\Validation\Rules\Unique.php
V5.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注入。