fujiokayu / S4

simple serverless secure storage
MIT License
1 stars 0 forks source link

Needs Fix: Upload / Download file directory traversal Vulnerability #21

Closed fujiokayu closed 3 years ago

fujiokayu commented 3 years ago

ファイルアップロードやダウンロードの際に Admin SDK で実装した特権的な API を使ってアクセスするフォルダを強制しているが、ファイル名によってはダウンロード時に

../targetUID/secret.txt

といったハックをされたり、アップロード時に

../targetUID/malware.exe

といった攻撃をされる可能性がある。

これはファイル名のサニタイズを行うことで容易に対応ができるが、サニタイズ処理に不備(ライブラリの脆弱性も含む)が混入する可能性も考えると、ファイルアクセスはフロントエンドで実装して Cloud Storage のセキュリティ・ルールに従わせる方針に転換した方が良いだろう。

fujiokayu commented 3 years ago

Fixed in 8ed12c6630241471f590d90d533caa1fc8438d7f and ddf3f72d668ee4c520f5059e4714e31382f3db2c