Unsynchronized get method, synchronized set method

Fierozen / owasp-esapi-java

Automatically exported from code.google.com/p/owasp-esapi-java

Other

0 stars 0 forks source link

Unsynchronized get method, synchronized set method #316

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago

I have V2.1 sources.

The class ReferenceEncryptedProperties.java has a synchronized setter and one 
of the two getters is also synchronized.  The second setter should also be 
synchronized for well behaved synchronization.

The corrected method signature follows:

public synchronized String getProperty(String key, String defaultValue) throws 
EncryptionRuntimeException {

Original issue reported on code.google.com by eamonn.w...@gmail.com on 26 Nov 2013 at 7:57

GoogleCodeExporter commented 9 years ago

Eamonn,
Would you like this work to be considered for the ESAPI hackathon contest? If 
so, please email me ASAP. Thanks.
-kevin wall <kevin.w.wall@gmail.com>

Original comment by kevin.w.wall@gmail.com on 23 Jan 2014 at 6:46