node-mapnik icon indicating copy to clipboard operation
node-mapnik copied to clipboard

Re-enable leak checking in address sanitizer builds

Open springmeyer opened this issue 8 years ago • 1 comments
trafficstars

Currently the leak sanitizer (part of address sanitizer) is reporting a number of leaks. They look maybe spurious to me? Maybe the would go away if mapnik itself were instrumented or if libc++ were instrumented as well? So the next actions here are to:

  • [ ] Instrument mapnik/libc++
  • [ ] Build node against instrumented libc++
  • [ ] Re-enable the leak checking that was disabled in https://github.com/mapnik/node-mapnik/commit/cd5f6b419ca447840b67cf26c10e050de260b91b
  • [ ] Evaluate if any of the remaining leaks are serious and need fixing

springmeyer avatar Apr 05 '17 22:04 springmeyer

For reference, here are the errors coming from the leak sanitizer (from a branch that I re-enabled it on): https://travis-ci.org/mapnik/node-mapnik/jobs/219065861

springmeyer avatar Apr 05 '17 22:04 springmeyer