http://bugzilla.slf4j.org/show_bug.cgi?id=176 --- Comment #25 from Simon Arlott <bugzilla.slf4j.simon@arlott.org> --- (In reply to comment #24)
Created attachment 101 [details] Patch for thread-safety issues during binding
The patch uses the same test previously attached. The implementation is slightly different than the other patch. I don't think the reentrancy checking should be necessary, but I don't know if other bindings actually invoke this method again.
// If re-entrant, return the temp factory
You're making an assumption that it will be re-entrant using the same thread. There is no guarantee that this is the case. The binding may be calling arbitrary external code that blocks while it does some initialisation. This initialisation may be performed with multiple threads in parallel, and one of those threads may try to log something. -- You are receiving this mail because: You are the assignee for the bug.