画像ファイルのアップロード機能について（PHP）

Question

画像ファイルをアップロードするPHPのスクリプトを作っています。

現在、セキュリティ的な観点から下記のような施策を行っていますが、他にすべきことはありますでしょうか。

// 拡張子「jpg」「jpeg」「gif」「png」のファイルのみ許可する
$ext = strrchr($file, ".");
if ( !in_array($ext,array(".jpg",".jpeg",".gif",".png")) ) {
  die("この拡張子のファイルは対応していません。画像ファイルをアップロードしてください。");
}

// ファイルのMIMEタイプを制限する
$pos = strpos($_FILES['myfile']['type'],"image");
if ( $pos === false ) {
  die("許可されていないファイル形式です。");
}
$arr = getimagesize($_FILES['myfile']['tmp_name']);
if ( ($arr["mime"] != "image/gif") && ($arr["mime"] != "image/jpeg") && ($arr["mime"] != "image/jpg") && ($arr["mime"] != "image/png") ) {
    die("許可されていないファイル形式です。");
}
if ( substr_count($_FILES['myfile']['type'], "/") > 1 ) {
    die("許可されていないファイル形式です。");
}

// アップロード後、ファイル名を変更する
$new_name = $upload_dir . md5(basename($_FILES['myfile']['name'])) . $ext;
if ( move_uploaded_file($_FILES['myfile']['tmp_name'], $new_name) ) {
    print "<img src=\"./temp" . $file . "\">";
} else {
    die("エラーが発生しました");
}

Answer 1

そうです。まず必ずmove_uploaded_file()を使用する、アップロードされたファイルのアクセス範囲は限定しておくなどは最低限やっておくべきです。

より万全を目指すならば、アップロードされた画像をGDライブラリやimagemagickで再加工することでしょうか。
gifでアップロードされた画像をpngに変換してさらにgifに戻すなどの処理で危険性はかなり回避できるのではないかと思います。