変数で作ったinclude パス
なぜ危険か
変数で作ったincludeパスは、その変数にユーザー入力が絶対に入らない場合だけ安全です。上流のコードでリクエスト値が入っていることが多いので確認しましょう。
include $template;$allowed = ['list', 'detail'];
if (!in_array($template, $allowed, true)) {
$template = 'list';
}
include __DIR__ . '/templates/' . $template . '.php';対策
変数に既知のファイル名しか入らないか確認し、__DIR__ と固定パスを使いましょう。
コードを検査する
コードを検査する →
ファイルを貼り付けると、このルールに該当する行をすべて示します。