extphprs / ext-php-rs / 23743442514

66%

master: 66%

Build Type

push

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().

Reduces valgrind-reported leaks by ~44% (10KB / 225 blocks) on the test
extension. Remaining leaks are from class registration — a separate concern.

Closes #540

Run Details

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

5 existing lines in 2 files now uncovered.

8071 of 12304 relevant lines covered (65.6%)

31.89 hits per line