isLoaded() and get() require a read lock only, but they call operator[] on _paths, which is a write operation. Since c++ maps are not thread-safe, this could cause memory corruption. The read-only members of DatabaseHolder should be made const functions if they can truly be implemented as read-only. (This would make accessing _paths.operator[] a compile error.)