hg: jdk8/tl/jdk: 7103570: AtomicIntegerFieldUpdater does not work when SecurityManager is installed (original) (raw)

Chris Hegarty chris.hegarty at oracle.com
Tue May 8 13:10:42 UTC 2012

On 08/05/2012 11:30, David Holmes wrote:

....

I was forgetting there's another major consideration here too, and that is that this code sync's up with Doug Lea's JSR-166 CVS repository. The changes I made are independent of the JDK used, but if I refactored things into ReflectUtil then the code would only be valid on a JDK with that update - which means Doug would need to maintain a different version of this fix for older JDKs.

Right, unless there is a good reason we should try not to maintain a separate fork here. Sync'ing is already a pain!

-Chris.

David -----

David -----

-Ulf

Am 08.05.2012 09:36, schrieb david.holmes at oracle.com: Changeset: 48513d156965 Author: dholmes Date: 2012-05-08 02:59 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/48513d156965 7103570: AtomicIntegerFieldUpdater does not work when SecurityManager is installed Summary: Perform class.getField inside a doPrivileged block Reviewed-by: chegar, psandoz ! src/share/classes/java/util/concurrent/atomic/AtomicIntegerFieldUpdater.java

! src/share/classes/java/util/concurrent/atomic/AtomicLongFieldUpdater.java ! src/share/classes/java/util/concurrent/atomic/AtomicReferenceFieldUpdater.java + test/java/util/concurrent/atomic/AtomicUpdaters.java

More information about the core-libs-dev mailing list