RFR: 8325179: Race in BasicDirectoryModel.validateFileCache

[Andrey Turbanov]

On Mon, 4 Mar 2024 20:21:30 GMT, Alexey Ivanov <aivanov at openjdk.org> wrote:

> Ensure access to the `filesLoader` field of `BasicDirectoryModel` is synchronized.
> 
> Without synchronization, a thread checks if `filesLoader` is not null and creates a new `FilesLoader` thread. If the thread is pre-empted between these two operations, another thread or even several threads can see the `null` value and create new `FilesLoader` threads.
> 
> The same way, `BasicDirectoryModel.invalidateFileCache` needs to be synchornized to interrupt the current `filesLoader` and assign `null`.
> 
> This bug allows reproducing `ConcurrentModificationException` seen in [JDK-8323670](https://bugs.openjdk.org/browse/JDK-8323670) and [JDK-8307091](https://bugs.openjdk.org/browse/JDK-8307091) using the test in PR #18109.

src/java.desktop/share/classes/javax/swing/plaf/basic/BasicDirectoryModel.java line 539:

> 537:             int remSize = (remFiles == null) ? 0 : remFiles.size();
> 538:             int addSize = (addFiles == null) ? 0 : addFiles.size();
> 539:             synchronized(fileCache) {

Suggestion:

            synchronized (fileCache) {

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/18111#discussion_r1512452216