(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.