php / php/php-src

FPM: Consider escaping script in status

オープン
#11,464 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

Bug SAPI: fpm
主要言語
C
スター
40.4k
フォーク
8.2k
平均マージ
2日 13時間
マージ済み PR(30日)
96

説明

Description

Currently the script field in status is not escaped for neither JSON, HTML nor XML. It should be investigated if any of the characters requiring escape can get to the script filename. This is very unlikely issue in practice but potentially theoretical possible so it might be a good idea to address it.

The thing to try would be to have a script whose path needs to be escaped. The server might be able to access with URL escape that is eventually transformed to the path with a character requiring an escape. This needs some testing if it's even possible.

PHP Version

Any with FPM

Operating System

Linux

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

まず、エスケープが必要な文字をパスに含むスクリプトを使って PHP-FPM のステータス出力をテストし、JSON、HTML、XML を対象にします。URL エスケープによってそのようなパスを生成できるか、また script フィールドが安全に出力されるかを確認します。完了とは、動作が確認され、必要なエスケープに対処されていることを意味します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
php
領域
backend
issue の種類
バグ
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。