( )⚙ D4152 index: use PyMem_Free() to free nodeetree instance

This is an archive of the discontinued Mercurial Phabricator instance.

index: use PyMem_Free() to free nodeetree instance
ClosedPublic

Authored by martinvonz on Aug 7 2018, 2:26 AM.

Details

Summary

As Yuya pointed out in the review of D4108, PyMem_Malloc() and
PyMem_Free() should be paired. IIUC, PyMem_Malloc() may use a
different allocator than malloc(), so using free() with a pointer from
PyMem_Malloc() may be very wrong.

Diff Detail

Repository
rHG Mercurial
Lint
Lint Skipped
Unit
Unit Tests Skipped

Event Timeline

martinvonz created this revision.Aug 7 2018, 2:26 AM
This revision was automatically updated to reflect the committed changes.