Hi,

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
attached?

Thanks, Clemens

Attachment: jdk.patch
Description: Binary data

Reply via email to