join() may only be called on a given thread once; calling it multiple times or simultaneously results in a fatal error. The construction currently used (if joinable then join) is not synchronized and thus racy. Simply locking the current mutex around this call will not work because it will deadlock, but adding another mutex just for this might be fine.