https://issues.apache.org/bugzilla/show_bug.cgi?id=41214

--- Comment #49 from msm <[email protected]> ---
(In reply to comment #47)
>             synchronized(c) {
>                 aai = c.aai;
>                 additive = c.additive;
>                 c = c.parent();
>             }
> 
>             if(aai != null) {
>                 writes += aai.appendLoopOnAppenders(event);
>             }

The code above is risky. The reason for that synchronized block is lost. That
code doesn´t protect against a simultaneous call of aai.appendLoopOnAppenders()
from two or more threads. You are throwing away the main purpose of that
synchronization.

The solution proposed in Comment 37 works pretty well, but unfortunately it
depends on JRE 1.5 or later. The solution proposed by me also works, even using
JRE 1.4.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to