extphprs / ext-php-rs / 23743465206

66%

master: 66%

Build Type

Pull #707

github

Committed by ptondereau

Commit Message

fix(memory): eliminate module definition memory leak

Store the ModuleEntry in a static (StaticModuleEntry) instead of
Box::into_raw, mirroring how C extensions use a static zend_module_entry.
Free all secondary heap allocations (name, version, functions, arg_info,
class-name pointers) during MSHUTDOWN via cleanup_module_allocations().

Cleanup only runs for shared extensions (handle != NULL) to avoid
use-after-free in embed SAPI where PHP still references module pointers
during later shutdown phases.

Closes #540

Pull Request Pull Request #707: fix(memory): eliminate module definition memory leak

Run Details

6 of 62 new or added lines in 1 file covered. (9.68%)

54 existing lines in 5 files now uncovered.

8071 of 12304 relevant lines covered (65.6%)

31.89 hits per line