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)

Reply via email to