New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Unsynchronized get method, synchronized set method #316

Open

GoogleCodeExporter opened this issue May 5, 2015 · 1 comment

Labels

auto-migrated Priority-Medium Type-Defect

GoogleCodeExporter commented May 5, 2015

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 [email protected] on 26 Nov 2013 at 7:57

The text was updated successfully, but these errors were encountered:

Author

GoogleCodeExporter commented May 5, 2015

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

Original comment by [email protected] on 23 Jan 2014 at 6:46

GoogleCodeExporter added Priority-Medium Type-Defect auto-migrated labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment