extphprs / ext-php-rs / 23741464792

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

46 of 65 new or added lines in 1 file covered. (70.77%)

3 existing lines in 1 file now uncovered.

8111 of 12307 relevant lines covered (65.91%)

31.89 hits per line