search

eddelbuettel / bh

R package providing Boost Header files
85 stars 33 forks source link

Container's pair.hpp triggers runtime error with gcc-UBSAN #37

Closed Enchufa2 closed 6 years ago

Enchufa2 commented 7 years ago

For instance, here:

> library(simmer)
/.../R/test-3.4/BH/include/boost/container/detail/pair.hpp:85:30: runtime error: reference binding to null pointer of type 'const struct piecewise_construct_t'
/.../R/test-3.4/BH/include/boost/container/detail/pair.hpp:85:30: runtime error: reference binding to null pointer of type 'const struct piecewise_construct_t'
/.../R/test-3.4/BH/include/boost/container/detail/pair.hpp:85:30: runtime error: reference binding to null pointer of type 'const struct piecewise_construct_t'
/.../R/test-3.4/BH/include/boost/container/detail/pair.hpp:85:30: runtime error: reference binding to null pointer of type 'const struct piecewise_construct_t'

Solved since v1.63.0. It is a very minor issue, because the offending pointer is never used (see ticket 12577), so no hurries at all. I just want to put this on the record, just in case someone struggles with the same issue.

eddelbuettel commented 7 years ago

Thank you -- agree completely. SAN/UBSAN is very useful, and the Boost codebase is so large that a few of these are lurking.

Referencing it here is the right idea, and we will get it fixed with 1.63, or maybe 1.64. No real hurry right now.

Enchufa2 commented 6 years ago

Issue fixed in the last update. We can close this. Thanks!

eddelbuettel commented 6 years ago

Awesome! Thanks for the follow-up.