Lazy object reset with object destructor can lead to use-after-free
維護者通常 1 天內回覆
還沒有人認領這個 Issue。
評估
研究方向
Start by reading Zend/zend_lazy_objects.c around zend_object_make_lazy at line 338, then compare the resetAsLazyGhost() and resetAsLazyProxy() paths with the related re-entrant destruction issue #22399. Reproduce with the supplied PHP snippet under AddressSanitizer; done means both reset methods avoid the heap-use-after-free.
由索引模型根據 Issue 內容生成。
描述
Description
Originally reported by @cnwangjihe.
The re-entrant property destruction issue reported in #22399 also affects ReflectionClass::resetAsLazyGhost() and ReflectionClass::resetAsLazyProxy().
The following code:
<?php
class Target {
public ?Trigger $trigger = null;
}
class Trigger {
public function __construct(private object $target) {}
public function __destruct() {
unset($this->target->trigger);
}
}
$reflection = new ReflectionClass(Target::class);
$target = new Target();
$target->trigger = new Trigger($target);
$reflection->resetAsLazyGhost($target, static function (): void {});
Resulted in this output:
ERROR: AddressSanitizer: heap-use-after-free
zend_object_make_lazy Zend/zend_lazy_objects.c:338
But I expected this output instead:
PHP Version
PHP 8.4+
Operating System
No response
- 主要語言
- C
- 星號
- 40.7k
- 分支
- 8.3k
- 平均合併
- 2 天 20 小時
- 30 天內合併 PR
- 147
環境準備
- 沒有 Dockerfile 或 Docker Compose 檔案
- 沒有 Pull Request 範本
- 閱讀貢獻指南
從這裡開始
- 先讀完整個 Issue，再讀專案的貢獻指南。
- 在 Issue 下留言說明你要接手 —— 這能避免兩個人做同樣的事。
- Fork 儲存庫，在一個分支上完成修改。
- 送出 Pull Request，並在描述裡引用這個 Issue 編號。
php/php-src 的其他 Issue
-
sapi/fpm/tests/tester.inc can send SIGUSR2 to pid 0, killing run-tests.php可能已有人在做 關聯的 PR 仍在進行中或已合併。 未關閉Bug Status: Needs Triage
難度 2/5 1-3 小時 新手友好度 78/100
維護者通常 1 天內回覆
-
zend_set_local_var() overwrites existing compiled variable (CV) zval without calling zval_ptr_dtor()可能已有人在做 關聯的 PR 仍在進行中或已合併。 未關閉Bug Status: Needs Triage
難度 2/5 1-3 小時 新手友好度 82/100
維護者通常 1 天內回覆
-
難度 2/5 1-3 小時 新手友好度 72/100
維護者通常 1 天內回覆
-
Case-insensitive substr_compare, ksort, krsort are still locale-dependent可能已有人在做 @lazerg 於 4 天前認領。 未關閉Bug Extension: standard Status: Verified
難度 2/5 1-3 小時 新手友好度 73/100
維護者通常 1 天內回覆
-
master: part of the mbstring bailout fix got lost in a merge可能已有人在做 @lazerg 於 4 天前認領。 未關閉
難度 2/5 1-3 小時 新手友好度 78/100
維護者通常 1 天內回覆
相似的 Issue
-
難度 2/5 1-3 小時 新手友好度 65/100
-
難度 2/5 1-3 小時 新手友好度 78/100
HexaCluster/credcheck#86 ·
-
難度 2/5 1-3 小時 新手友好度 66/100
mongodb/winkerberos#175 ·
-
bug
難度 2/5 1-3 小時 新手友好度 74/100
AOMediaCodec/SVT-AV1#3320 ·
-
difficulty: easy
難度 2/5 1-3 小時 新手友好度 64/100