hg: jdk8/tl/jdk: 7103570: AtomicIntegerFieldUpdater does not work when SecurityManager is installed

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