[validator] ValidatorAction needs thread-safe

• Key: VALIDATOR-89
• Type: Bug
• Status: Closed
• Resolution: Fixed
• Priority: Major
• Assignee: Unassigned
• Reporter: Takayuki Kaneko
• Fix Version/s: 1.3.1 Release

Environment

Operating System: All
Platform: All

Description

ValidatorAction needs thread-safe because it is cached by other programs, like
Struts.
But ValidatorAction has an unthread-safe block.

Here is a patch below.

• • • ValidatorAction.java Mon Apr 24 22:41:55 2006
    • ValidatorAction.java.new Mon Apr 24 22:44:54 2006
      ***************
    • 527,536 ****
      params.put(Validator.VALIDATOR_ACTION_PARAM, this);

try {
! ClassLoader loader = this.getClassLoader(params);
! this.loadValidationClass(loader);
! this.loadParameterClasses(loader);
! this.loadValidationMethod();

Object[] paramValues = this.getParameterValues(params);

— 527,540 ----
params.put(Validator.VALIDATOR_ACTION_PARAM, this);

try {
! if (this.validationMethod == null) {
! synchronized(this) { ! ClassLoader loader = this.getClassLoader(params); ! this.loadValidationClass(loader); ! this.loadParameterClasses(loader); ! this.loadValidationMethod(); ! }
! }

Object[] paramValues = this.getParameterValues(params);