I personal main issue is with the SubstitueLoggers in general. I can almost guarantee I'll see a bug report if the substitute logger messages show up in a users log.

At a glance though this does look safe to me since you synchronize on the class itself.