Found this small bug while poaching code for another project ;) It
presumably would cause an IndexOutOfBoundsException when comparing
zero-length strings.

--Jeff
Index: src/java/org/apache/commons/latka/validators/GoldenFileValidator.java
===================================================================
RCS file: 
/home/cvspublic/jakarta-commons/latka/src/java/org/apache/commons/latka/validators/GoldenFileValidator.java,v
retrieving revision 1.4
diff -u -r1.4 GoldenFileValidator.java
--- src/java/org/apache/commons/latka/validators/GoldenFileValidator.java       
2001/09/07 19:45:45     1.4
+++ src/java/org/apache/commons/latka/validators/GoldenFileValidator.java       
+2001/12/22 05:21:24
@@ -149,7 +149,7 @@
 
     for (int i=0; i<str1.length() ; i++ ) {
       if (str1.charAt( i ) != str2.charAt( i ) ) {
-        log("Error at " + i  + " " + str1.charAt(1) +
+        log("Error at " + i  + " " + str1.charAt(i) +
             str2.charAt(i));
         return false;
       }
Index: src/java/org/apache/commons/latka/validators/GoldenFileValidator.java
===================================================================
RCS file: 
/home/cvspublic/jakarta-commons/latka/src/java/org/apache/commons/latka/validators/GoldenFileValidator.java,v
retrieving revision 1.4
diff -u -r1.4 GoldenFileValidator.java
--- src/java/org/apache/commons/latka/validators/GoldenFileValidator.java       
2001/09/07 19:45:45     1.4
+++ src/java/org/apache/commons/latka/validators/GoldenFileValidator.java       
+2001/12/22 05:21:24
@@ -149,7 +149,7 @@
 
     for (int i=0; i<str1.length() ; i++ ) {
       if (str1.charAt( i ) != str2.charAt( i ) ) {
-        log("Error at " + i  + " " + str1.charAt(1) +
+        log("Error at " + i  + " " + str1.charAt(i) +
             str2.charAt(i));
         return false;
       }

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to