Hi all,

This commit fixes a bug whereby FilePermission("/") did not imply
itself.  There was some old (and broken) code for compensating for
paths with trailing separators.  Nowadays paths are canonicalized;
canonical paths never have trailing separators, so I just junked
the old code.

Cheers,
Gary
Index: ChangeLog
===================================================================
RCS file: /cvsroot/classpath/classpath/ChangeLog,v
retrieving revision 1.7965
diff -u -r1.7965 ChangeLog
--- ChangeLog   28 Jun 2006 20:10:25 -0000      1.7965
+++ ChangeLog   29 Jun 2006 09:01:35 -0000
@@ -1,3 +1,7 @@
+2006-06-29  Gary Benson  <[EMAIL PROTECTED]>
+
+       * java/io/FilePermission.java (implies): Work when path is "/".
+
 2006-06-28  Andrew John Hughes  <[EMAIL PROTECTED]>
 
        * java/lang/Thread.java:
Index: java/io/FilePermission.java
===================================================================
RCS file: /cvsroot/classpath/classpath/java/io/FilePermission.java,v
retrieving revision 1.21
diff -u -r1.21 FilePermission.java
--- java/io/FilePermission.java 29 Mar 2006 15:33:24 -0000      1.21
+++ java/io/FilePermission.java 29 Jun 2006 09:01:35 -0000
@@ -274,12 +274,7 @@
        break;
 
       default:
-       if (f2.charAt(f2.length() - 1) == File.separatorChar)
-         {
-           if (! f1.equals(f2.substring(0, f2.length() - 1)))
-             return false;
-         }
-       else if (!f1.equals(f2))
+       if (!f1.equals(f2))
          return false;
        break;
       }

Reply via email to