php / php/php-src

`restore_xxx_handler()` does silently return even if there is no handler to restore

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

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

Feature Status: Needs Triage
主要言語
C
スター
40.4k
フォーク
8.1k
平均マージ
2日 13時間
マージ済み PR(30日)
96

説明

Description

The following code:

repro: https://3v4l.org/DDulh

<?php

restore_error_handler();
restore_error_handler();
restore_error_handler();

restore_exception_handler();
restore_exception_handler();
restore_exception_handler();

Resulted in this output:

But I expected this output instead:

php warning for each restore_xxx_handler() call if there is no handler to restore

The warnings are important to be inform the developer the operation is probably not wanted (ie. handler was never added).

PHP Version

any

Operating System

any

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

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

はじめの一歩

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

調査の方向性

まず PHP ソース内の restore_error_handler() と restore_exception_handler() の実装を特定し、繰り返し呼び出しが現在どのように処理されているかを調べます。報告されたシーケンスを再現し、復元する handler がない状態で行われる呼び出しのカバレッジを追加します。それぞれの呼び出しが期待される PHP 警告を発生させ、通常の handler の復元を壊さなければ完了です。

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

評価

技術スタック
c, php
領域
backend
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
42/100

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

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