Details
Major Description
It tries to access STL containers which may be used by other threads without locking them. These STL containers could change causing the stats code to dereference illegal memory.
(the connections stats group was initially added as a "last resort" to dump information from an unstable system and not intended to be run in production. Given that it looks like people are using it we should fix that). I think we should continue to do dirty reads to avoid potential deadlocks.