https://bz.apache.org/bugzilla/show_bug.cgi?id=58099
--- Comment #9 from Philippe Mouawad <[email protected]> --- (In reply to Sebb from comment #7) > Comment on attachment 33567 [details] > Patch fixing this issue > > [New file needs an AL header] fixed > > Looks like adaptee is created using double-checked locking. > That is not thread-safe unless protected by volatile (or a full lock, in > which case no point in double-checking). > Thanks, used volatile. > Given that the adaptee depends only on the value of CPS_HTTPS, and that is > only checked at startup, there will only ever be one adaptee. So the code > could perhaps use the IODH idiom instead. Didn't do it, feel free to do it if you think it is useful. -- You are receiving this mail because: You are the assignee for the bug.
