フォームで送られてきた画像を保存する際、同じ名前だったら上書きをする

Question

フォームからファイルを送信して、もし既に存在するファイル名だったら上書きをしたいんですが、どうすればいいですか？

$path = sprintf($_SERVER['DOCUMENT_ROOT'] . './users/'.$_POST['keyid'].".png");
if (!move_uploaded_file($_FILES['upfile']['tmp_name'], $path)) {
	throw new RuntimeException('ファイル保存時にエラーが発生しました');
}
chmod($path, 0644);

処理の一部なんですが、if (!move_uploaded_file($_FILES['upfile']['tmp_name'], $path)) {でエラーが出てるみたいです。
でも、ここを存在しない名前だといけるんで、ファイルが合った場合は上書きをしたいんです。どうすればいいですか？

Comments

[退会済みユーザー]

同じ画像じゃなくても名前が一緒だったら上書きしちゃうの？

[ryo-dev]

はい。上書きします。

[退会済みユーザー]

と言うことは、画像ファイルをエクセルで上書きすることもできちゃいますね。

[ryo-dev]

どういう意味でしょうか？

[m.ts10806]

test.pngという名前の画像がすでにあったとしてエクセルファイルをtest.pngと名前を変更してアップロードされたとしても上書きしていいの？ってことです。名前は画像っぽいけど実態はエクセル（要は画像じゃないってこと）

[ryo-dev]

なるほど、要するに、ファイルの内容が画像データじゃなかったら、エラーを出して、処理を止める必要があるということですか？

[m.ts10806]

画像を表示したいけど実態は画像じゃないから表示できませんし、幾らでも悪さができますよね。その辺りの概念とかチェックの仕方は別途調べてください。幾らでも出てきます。

[ryo-dev]

なるほど、ありがとうございます。ちょっとエンドユーザを信用しすぎてました。拡張子だけで判断するのはよくないということですね。

[m.ts10806]

ユーザからの入力は信用しない　が基本です。もちろん社内システムのような限定された環境であっても。

[ryo-dev]

勉強になります。

Answer 1

$_SERVER['DOCUMENT_ROOT']は文字の末尾にスラッシュがつかないので

$_SERVER['DOCUMENT_ROOT'] . './users/'

とすると、最後のディレクトリ名のケツに「.」がついて（おそらく）
存在しないディレクトリに書き込もうとしませんか？

またユーザーから送られてきた「$_POST['keyid']」でファイル名を決めると
いうのはセキュリティ的にあまりよろしくないと思います
サーバー上に書き込まれるデータの名称はなるべくサーバー側から与えてやるべきでは？

Comments

[ryo-dev]

サーバに書き込むファイル名は、サーバから与えた方が良いですね！

Answer 2

move_uploaded_file()はもし既にファイルがあれば上書きするはずですので、パーミッションの問題なのかなぁと思います。

$path = sprintf($_SERVER['DOCUMENT_ROOT'] . './users/'.$_POST['keyid'].".png");

//既存ファイルのパーミッションを書き込み可にする
if(file_exists($path)){
chmod($path, 0664);
}

//それかファイルがあれば削除する。もしかしたらこっちの方が明快？
/*
if(file_exists($path)){
 unlink($path);
}
*/

if (!move_uploaded_file($_FILES['upfile']['tmp_name'], $path)) {
    throw new RuntimeException('ファイル保存時にエラーが発生しました');
}
chmod($path, 0644);

Comments

[ryo-dev]

ありがとうございます。
解決できました。