<AWT Dev> numAppContexts in AppContext modified in not-thread-safe way.

Clemens Eisserer linuxhippy at gmail.com
Thu Aug 18 09:28:33 PDT 2011


While looking for concurrency issues in caciocavallo-web, I found the
numAppContexts field in AppContext is used in a non-thread-safe way.
Read-modify-write operations aren't atomic on volatile fields.

What do you think about replacing it with an AtomicInteger, as in the patch

Thanks, Clemens
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20110818/21ee57fe/attachment.html 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: jdk.patch
Type: application/octet-stream
Size: 2013 bytes
Desc: not available
Url : http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20110818/21ee57fe/jdk.patch 

More information about the awt-dev mailing list