“典型的やられサイト”で学ぶセキュリティのワナ

え？　PHPファイルを登録できるのはまずいでしょ……　Webアプリの「アップロード機能」に潜む“あるある”ワナ（3/3 ページ）

公開 2023年12月22日 15時00分

更新 2024年01月19日 12時23分

著者

谷井将人

[ITmedia]

印刷する Googleで優先するソースとして追加

上野　もう一つはレースコンディションだね。何回か前に会員登録機能の実装をやってもらったんだけど覚えているかな？

参考記事：え？　同じIDで登録できる？　コンビニ証明書で話題「同時処理」の危険性

荒井　はい。確認が甘いと、複数人が同じIDで登録できてしまったりと意図しない挙動が起きるという話がありました。

上野　今回のファイルアップロードでも似たようなことが起きたりする。複数人が同じ名前の異なるファイルを同時に登録すると、これもまた意図しない挙動が起きる可能性がある。

上野　例えば、AさんとBさんが同時に「戸籍証明書.PDF」をアップロードしたとしよう。この戸籍証明書.PDFは同じファイル名だけど、AさんBさんそれぞれの戸籍証明書だから中身は別だ。これを実際に同時にアップロードすると……できた。Aさんは自分の戸籍証明書.PDFを開いたつもりなのに、Bさんの戸籍証明書が表示されてしまった。

荒井　情報漏えいだ！

上野　そうだね。どこかで意図しない挙動が起きて、AさんのデータがBさんのデータに上書きされているようだ。これも直しておこう。

荒井　ダウンロードには問題ないですか？

上野　実はここまでの問題って、ダウンロード機能があるから発生しているんだよね。IT業界にいる人なら分かると思うけど、ここまでに出てきた「添付ファイルを開く」「戸籍証明書.PDFを開く」という操作って「ダウンロードする」とは言っていないけど、データのダウンロードをしているよね。

荒井　ダウンロードしないと開けないですもんね。

上野　そう。「開く」という操作をしなければPHPコマンドは実行されなかったし、他人の戸籍証明書を誰かが見ることもなかった。アップロードとダウンロードは一体なんだよね。

荒井　よく分かりました。では、指摘箇所直してきます！

上野　頑張れ～～。

──しばらくして……

荒井　できました！

上野　OK！　じゃあ次回はいよいよ最終回だよ。気を引き締めていこう。