Fix ValidSafeHTML bean validator

aabashkin · aabashkin · commit e7c61c41a77a · 2026-01-05T16:02:25.000-05:00
diff --git a/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidSafeHTMLValidator.java b/src/main/java/org/owasp/esapi/reference/validation/annotations/ValidSafeHTMLValidator.java
@@ -26,7 +26,8 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida
             return true;
         }
         ValidationErrorList errorList = new ValidationErrorList();
-        boolean valid = ESAPI.validator().isValidSafeHTML(context, input, maxLength, allowNull, errorList);
+        ESAPI.validator().getValidSafeHTML(context, input, maxLength, allowNull, errorList);
+        boolean valid = errorList.isEmpty();
         
         if(!valid){
             ValidationUtil.addViolations(errorList, constraintValidatorContext);

Original file line number	Diff line number	Diff line change
`@@ -26,7 +26,8 @@ public boolean isValid(String input, ConstraintValidatorContext constraintValida`
`26`	`26`	`return true;`
`27`	`27`	`}`
`28`	`28`	`ValidationErrorList errorList = new ValidationErrorList();`
`29`		`- boolean valid = ESAPI.validator().isValidSafeHTML(context, input, maxLength, allowNull, errorList);`
	`29`	`+ ESAPI.validator().getValidSafeHTML(context, input, maxLength, allowNull, errorList);`
	`30`	`+ boolean valid = errorList.isEmpty();`
`30`	`31`
`31`	`32`	`if(!valid){`
`32`	`33`	`ValidationUtil.addViolations(errorList, constraintValidatorContext);`