ファイルシステムのセキュリティ

PHP は、ファイルおよびディレクトリ毎に権限を設定する多くのサーバーシステム上に組み込まれたセキュリティを提供します。これにより、ファイルシステム内のファイルを読み込み可能に制御することが可能になります。全てのファイルは世界中から読み込み可能であり、このファイルシステムにアクセスした全てのユーザーから読み込まれても安全であることを確認する必要があります。

PHPは、ファイルシステムにユーザーレベルのアクセスを許可するように設計されているため、PHPスクリプトから /etc/passwd のようなシステムファイルを読み込み可能としたり、イーサネット接続を修正したり、巨大なプリンタジョブを出力したりすることができます。これから明らかにわかることですが、読み書きするファイルを適切に設定する必要があります。

各自のホームディレクトリにあるファイルを削除する次のスクリプトを見てみましょう。これは、ファイル管理用にWebインターフェイスを使用する場合に通常生じるような設定を仮定しています。この場合、Apacheユーザはそのユーザーのホームディレクトリにあるファイルを削除可能です。

例1 甘い変数の確認から生じるリスク

<?php

// ユーザーのホームディレクトリからファイルを削除する
$username = $_POST['user_submitted_name'];
$userfile = $_POST['user_submitted_filename'];
$homedir  = "/home/$username";

unlink("$homedir/$userfile");

echo "ファイルは削除されました!";

?>

username および filename はユーザーフォームから投稿可能であるため、別の username および filename を投稿して削除すべきではない削除することが可能となります。この場合、他の何らかの形式の認証を使用するべきです。投稿された変数が、 "../etc/" と "passwd" であった場合について考えてみましょう。簡単なコードを以下に示します。

例2 ... ファイルシステムへの攻撃

<?php

// 外部からPHPユーザーがアクセス可能なハードドライブを削除します。PHPが
// ルートのアクセス権限を有している場合、
$username = $_POST['user_submitted_name']; // "../etc"
$userfile = $_POST['user_submitted_filename']; // "passwd"
$homedir  = "/home/$username"; // "/home/../etc"

unlink("$homedir/$userfile"); // "/home/../etc/passwd"

echo "ファイルは削除されました!";

?>

こうした問題を防止するために必要な重要なチェック手段として以下の2 種類のものがあります。

PHP Webユーザーバイナリに制限された権限のみを許可する。
投稿された全ての変数を確認する。

以下に改良されたスクリプトを示します。

例3 より安全なファイル名の確認

<?php

// PHPユーザーがアクセス可能なハードドライブからファイルを削除する。
$username = $_SERVER['REMOTE_USER']; // 認証機構を使用する
$userfile = basename($_POST['user_submitted_filename']);
$homedir  = "/home/$username";

$filepath = "$homedir/$userfile";

if (file_exists($filepath) && unlink($filepath)) {
    $logstring = "$filepath を削除しました\n";
} else {
    $logstring = "$filepath の削除に失敗しました\n";
}
j
$fp = fopen("/home/logging/filedelete.log", "a");
fwrite($fp, $logstring);
fclose($fp);

echo htmlentities($logstring, ENT_QUOTES);

?>

しかし、これでも、傷口を塞いだことにはなりません。ユーザーが自分用のユーザーログインを作成することをあなたの認証システムが許可しており、ユーザーが "../etc/" へのログインを選択した場合、システムはまたも公開されてしまいます。このため、よりカスタマイズされたチェックを行なう方がよいでしょう。

例4 より安全なファイル名の確認

<?php

$username     = $_SERVER['REMOTE_USER']; // 認証機構を使用する
$userfile     = $_POST['user_submitted_filename'];
$homedir      = "/home/$username";

$filepath     = "$homedir/$userfile";

if (!ctype_alnum($username) || !preg_match('/^(?:[a-z0-9_-]|\.(?!\.))+$/iD', $userfile)) {
    die("Bad username/filename");
}

// etc.

?>

オペレーティングシステムによって、注意するべきファイルは大きく変化します。これらには、デバイスエントリ(/dev/ または /dev/)、設定ファイル(/etc/ ファイルおよび .ini ファイル)、よく知られたファイル保存領域 (/home/、My Documents)等が含まれます。このため、明示的に許可するもの以外の全てを禁止する方針とする方が通常はより簡単です。

Found A Problem?

Learn How To Improve This Page • Submit a Pull Request • Report a Bug

＋add a note

User Contributed Notes 5 notes

down

anonymous ¶

19 years ago

(A) Better not to create files or folders with user-supplied names. If you do not validate enough, you can have trouble. Instead create files and folders with randomly generated names like fg3754jk3h and store the username and this file or folder name in a table named, say, user_objects. This will ensure that whatever the user may type, the command going to the shell will contain values from a specific set only and no mischief can be done.

(B) The same applies to commands executed based on an operation that the user chooses. Better not to allow any part of the user's input to go to the command that you will execute. Instead, keep a fixed set of commands and based on what the user has input, and run those only. 

For example,
(A) Keep a table named, say, user_objects with values like:
username|chosen_name   |actual_name|file_or_dir
--------|--------------|-----------|-----------
jdoe    |trekphotos    |m5fg767h67 |D
jdoe    |notes.txt     |nm4b6jh756 |F
tim1997 |_imp_ folder  |45jkh64j56 |D

and always use the actual_name in the filesystem operations rather than the user supplied names.

(B)
<?php
$op = $_POST['op'];//after a lot of validations 
$dir = $_POST['dirname'];//after a lot of validations or maybe you can use technique (A)
switch($op){
    case "cd":
        chdir($dir);
        break;
    case "rd":
        rmdir($dir);
        break;
    .....
    default:
        mail("webmaster@example.com", "Mischief", $_SERVER['REMOTE_ADDR']." is probably attempting an attack.");
}

down

fmrose at ncsu dot edu ¶

19 years ago

All of the fixes here assume that it is necessary to allow the user to enter system sensitive information to begin with. The proper way to handle this would be to provide something like a numbered list of files to perform an unlink action on and then the chooses the matching number. There is no way for the user to specify a clever attack circumventing whatever pattern matching filename exclusion syntax that you may have.

Anytime you have a security issue, the proper behaviour is to deny all then allow specific instances, not allow all and restrict. For the simple reason that you may not think of every possible restriction.

down

devik at cdi dot cz ¶

23 years ago

Well, the fact that all users run under the same UID is a big problem. Userspace  security hacks (ala safe_mode) should not be substitution for proper kernel level security checks/accounting.

Good news: Apache 2 allows you to assign UIDs for different vhosts.

devik

down

Latchezar Tzvetkoff ¶

15 years ago

A basic filename/directory/symlink checking may be done (and I personally do) via realpath() ...



<?php



if (isset($_GET['file'])) {

    $base = '/home/polizei/public_html/';  // it seems this one is good to be realpath too.. meaning not a symlinked path..

    if (strpos($file = realpath($base.$_GET['file']), $base) === 0 && is_file($file)) {

        unlink($file);

    } else {

        die('blah!');

    }

}

?>

down

cronos586(AT)caramail(DOT)com ¶

22 years ago

when using Apache you might consider a apache_lookup_uri on the path, to discover the real path, regardless of any directory trickery.
then, look at the prefix, and compare with a list of allowed prefixes.
for example, my source.php for my website includes:
if(isset($doc)) {
    $apacheres = apache_lookup_uri($doc);
    $really = realpath($apacheres->filename);
    if(substr($really, 0, strlen($DOCUMENT_ROOT)) == $DOCUMENT_ROOT) {
        if(is_file($really)) {
            show_source($really);
        }
    }
}
hope this helps
regards,
KAT44

＋add a note

ファイルシステムのセキュリティ

目次

Found A Problem?

User Contributed Notes 5 notes