Robert Geiger created GEODE-1120:
------------------------------------
Summary: Synchronization on local variables in Message
Key: GEODE-1120
URL: https://issues.apache.org/jira/browse/GEODE-1120
Project: Geode
Issue Type: Improvement
Components: core
Reporter: Robert Geiger
gemfire-core/src/main/java/com/gemstone/gemfire/internal/cache/tier/sockets/Message.java
and extending classes (ChunkedMessage) have synchronization statements that
appear to have the purpose of making the comm buffer use safe for read/write
from different threads. The class however is clearly not thread safe, and
synchronization is done on local variables in multiple places and so is
incorrect.
If there is a defined purpose for the synchronization it should be done
correctly; if not it should be removed.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)