master: part of the mbstring bailout fix got lost in a merge
メンテナーはふだん 1 日以内に返信
評価
調査の方向性
ext/mbstring/libmbfl/mbfl/mbfl_encoding.hを読み、mb_convert_buf_ensure()のハンクをPHP-8.6と比較してください。参照されているマージdiffを調べ、どのように削除されたかを確認してください。PHP-8.6から欠落しているテストをext/mbstring/tests/oom_buf_growth.phptに復元し、そのテストを実行してください。masterに3つのバッファ更新とテストが含まれ、PHP-8.6と一致したら完了です。
索引モデルが issue の本文から書いたものです。
説明
The PHP-8.6 into master merge 9705fda1bb dropped this hunk from mb_convert_buf_ensure() in ext/mbstring/libmbfl/mbfl/mbfl_encoding.h. It also dropped the test ext/mbstring/tests/oom_buf_growth.phpt:
if (!newstr) {
+ /* An earlier growth may have moved the buffer. */
+ buf->out = *out;
+ buf->limit = *limit;
return FAILURE;
}
Both are on PHP-8.6 but missing on master. git diff origin/master origin/PHP-8.6 -- ext/mbstring/libmbfl/mbfl/mbfl_encoding.h shows only this hunk, and git show --remerge-diff 9705fda1bb -- ext/mbstring shows the merge removing it. I'm guessing it was an accident during conflict resolution. Re-applying the three lines and restoring the test from PHP-8.6 should be all that's needed.
- 主要言語
- C
- スター
- 40.7k
- フォーク
- 8.3k
- 平均マージ
- 2日 15時間
- マージ済み PR（30日）
- 155
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
php/php-src のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
Case-insensitive substr_compare, ksort, krsort are still locale-dependent対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンBug Status: Needs Triage
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
php/php-src#24552 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
HTTP wrapper leaks Location buffer when an empty Location header is followed by another Location header対応中かも @bercedev が 2 日前に担当しました。 オープンBug Category: Streams Status: Verified
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
`Time\Duration::__unserialize()` accepts a "negative zero" payload.対応中かも @TimWolla が今日担当しました。 オープンBug Status: Needs Triage
難易度 2/5 1〜3時間 初心者へのやさしさ 71/100
php/php-src#24505 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Bug Status: Needs Triage
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
メンテナーはふだん 1 日以内に返信
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 95/100
containers/crun#2415 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
zephyrproject-rtos/zephyr#123423 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
メンテナーはふだん 1 日以内に返信
-
dspromote: /etc/exports uses -alldirs on non-mount-point directories, so mountd exports nothingオープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 83/100
nextbsd/nextbsd-userland#434 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 77/100
メンテナーはふだん 1 日以内に返信