Memory leaks in GEOSSTRTree_nearest

[dbaston]

Output from Valgrind on unit tests:

16,480 (5,768 direct, 10,712 indirect) bytes in 103 blocks are definitely lost in loss record 15 of 16
operator new(unsigned long)
geos::geom::GeometryFactory::createPoint(geos::geom::CoordinateSequence*) const
geos::geom::GeometryFactory::createPoint(geos::geom::Coordinate const&) const
geos::geom::GeometryFactory::toGeometry(geos::geom::Envelope const*) const
geos::geom::Geometry::getEnvelope() const
GEOSSTRtree_nearest_r
GEOSSTRtree_nearest
void tut::test_object<tut::test_capistrtree_data>::test<1>()
tut::test_group<tut::test_capistrtree_data, 50>::run_test_seh_(void (tut::test_object<tut::test_capistrtree_data>::*)(), tut::test_group<tut::test_capistrtree_data, 50>::safe_holder<tut::test_object>&, std::__cxx11::basic_string<char, std::char_traits, std::allocator>&, int)
tut::test_group<tut::test_capistrtree_data, 50>::run_test_(std::_Rb_tree_iterator<std::pair> const&, tut::test_group<tut::test_capistrtree_data, 50>::safe_holder<tut::test_object>&, tut::test_result&)
tut::test_group<tut::test_capistrtree_data, 50>::run_next(tut::test_result&)
tut::test_runner::run_all_tests_in_group_(std::_Rb_tree_const_iterator<std::pair>) const

448 (64 direct, 384 indirect) bytes in 1 blocks are definitely lost in loss record 10 of 16
operator new(unsigned long)
geos::geom::GeometryFactory::createPolygon(geos::geom::LinearRing*, std::vector<geos::geom::Geometry*, std::allocator>*) const
geos::geom::GeometryFactory::toGeometry(geos::geom::Envelope const*) const
geos::geom::Geometry::getEnvelope() const

[strk]

See if it is appropriate to use getEnvelope_internal instead, which gets a link to the internal (Geometry-owned) envelope.

[dbaston]

Resolved in 679c005/git for 3.6, 6c94638/git in master