Boost C++ Libraries: Ticket #11533: rtree segfault / assertion failure

attachment set

gewesp@… — Tue, 11 Aug 2015 16:15:30 GMT

attachment → rtree-crash.cpp

owner changed; keywords set

awulkiew — Wed, 12 Aug 2015 23:00:28 GMT

keywords rtree remove assert added
owner changed from Barend Gehrels to awulkiew

Thanks for the report!

The removal doesn't work correctly when the min elements number is 1. I pushed a fix which you could apply to your local copy of Boost and check if it solves the issue for you: https://github.com/boostorg/geometry/commit/098279e7399efad87cc67d1e211b2b6a2e8495d8

Unfortunately it probably won't make it to Boost 1.59.

A workaround would be to define some greater number of min elements than 1, e.g.:

quadratic<4, 2>

gewesp@… — Thu, 13 Aug 2015 06:34:04 GMT

Thanks a lot for the quick fix (and for the library in the first place)! Can I also apply it to boost 1.55? That's the version I'm using in production.

Did you try it against the test case in rtree-crash.cpp? If it fixes that one, it very likely also fixes my production problem. I also suggest adding the test case to the test suite.

I'll try ASAP, on vacation right now.

status, version, milestone changed; resolution set

awulkiew — Thu, 13 Aug 2015 19:28:02 GMT

status new → closed
version Boost 1.58.0 → Boost 1.59.0
resolution → fixed
milestone To Be Determined → Boost 1.60.0

It should be possible to apply it to 1.55. In this case you only need to patch the remove visitor.

I tested develop branch (1.59+) against the code you sent. Test case was added as well.

I'm closing the ticket. If the problem was still there you could just reopen it.