extphprs / ext-php-rs / 25674249509

66%

master: 66%

Build Type

Pull #735

github

Committed by ptondereau

Commit Message

test: regression coverage for ZendClassObject<T> set_zval refcount leak

Adds an integration test that exercises the exact buggy code path
fixed in the previous commit: a class extending \Exception with a
#[php(prop)] field, constructed via ZendClassObject::new(...)
.into_zval(...) and thrown via PhpException::with_object. Memory
growth across 1024 throws is measured against a warm baseline:
the unfixed path leaks ~80 bytes per iteration (~80 KB total),
the fixed path stays within a few KB of allocator jitter.

The HT shutdown assertion only fires on a debug PHP build, so
measuring growth keeps the regression visible on release PHP
too. Verified locally: the test fails (655 KB growth) with the
fix reverted and passes (<8 KB growth) with the fix applied.

Pull Request Pull Request #735: fix: leak one refcount in ZBox<ZendClassObject<T>>::set_zval

Coverage Stats

0 of 4 new or added lines in 2 files covered. (0.0%)

9 existing lines in 1 file now uncovered.

8653 of 13065 relevant lines covered (66.23%)

33.26 hits per line