Details
-
Bug
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
Description
SoftRefFilesCache and DefaultFileMonitor use a boolean to communicate between threads, but don't use synchronisation.
There is no guarantee that the updated value of the boolean will be seen by the other thread. Adding volatile would fix this.
There are also various private instance fields that are not modified after construction.
If these were made final, it would improve thread-safety because the fields would then be published correctly across threads.