DefaultMessageHolder.java line 33
The messages field (line 33) is a plain ArrayList accessed and mutated from multiple public methods (addMessage, newMessage, render, countMessagesOfType, etc.) without any synchronization. In a multi-threaded environment this can lead to visibility issues and ConcurrentModificationException.
Bug #6
DefaultMessageHolder.javaline 33The
messagesfield (line 33) is a plainArrayListaccessed and mutated from multiple public methods (addMessage,newMessage,render,countMessagesOfType, etc.) without any synchronization. In a multi-threaded environment this can lead to visibility issues andConcurrentModificationException.