パスインジェクションを防ぐために$ _GET変数をどのようにサニタイズするのですか？

Question

$_GET['']リクエストをサニタイズする最善の方法は何ですか？ 1つのディレクトリからのみファイルをダウンロードできるようにしたい。

$baseDir = "/home/html/xy.com/public_html/downloads/";  
$path = realpath($baseDir . $_GET['file']); 

次のステップは何ですか。ここで

Answer 1

は、私はあなたがそこに持っている行の後にどうなるのかです：

基本的に

if (dirname($path) === $baseDir) { 
    //Safe 
} 

http://php.net/dirname

、ファイルはあなたがサポートし、1回のパスで実際に何かを送信する前にチェックを行います。 dirname()は末尾のパス区切り文字を残さないため、ファイル名の前に独自の/を追加して（$path）、$baseDir定義から削除する必要があります。

Answer 2

$_GETに$pathを割り当てる前に、値をサニタイズする必要があります。

PHPの独自のサニタイズ機能（http://php.net/manual/en/function.filter-input.phpおよびhttp://php.net/manual/en/filter.filters.sanitize.php）の使用をお勧めします。 PHP> = 5.2.0、

Answer 3

相対パスフラグメントが存在しないことを事後チェックするのではなく、すぐにストリッピングする方が簡単です。すでにそれはあなたのベースディレクトリから上方または下方に移動することができないことを保証

$baseDir = "/home/html/xy.com/public_html/downloads/";  
$path = realpath($baseDir . basename($_GET['file'])); 

：あなたは値をフェッチするときだけで、すぐにbasename()を使用しています。