CVSROOT: /cvsroot/classpath Module name: classpath Changes by: Raif S. Naffah <raif> 06/06/11 00:05:10
Modified files: . : ChangeLog gnu/javax/security/auth/login: ConfigFileParser.java Log message: 2006-06-11 Raif S. Naffah <[EMAIL PROTECTED]> * gnu/javax/security/auth/login/ConfigFileParser.java (validateClassName): Check that every component of 'cn' starts with a valid Java identifier char. CVSWeb URLs: http://cvs.savannah.gnu.org/viewcvs/classpath/ChangeLog?cvsroot=classpath&r1=1.7763&r2=1.7764 http://cvs.savannah.gnu.org/viewcvs/classpath/gnu/javax/security/auth/login/ConfigFileParser.java?cvsroot=classpath&r1=1.2&r2=1.3 Patches: Index: ChangeLog =================================================================== RCS file: /cvsroot/classpath/classpath/ChangeLog,v retrieving revision 1.7763 retrieving revision 1.7764 diff -u -b -r1.7763 -r1.7764 --- ChangeLog 10 Jun 2006 21:39:47 -0000 1.7763 +++ ChangeLog 11 Jun 2006 00:05:10 -0000 1.7764 @@ -1,3 +1,8 @@ +2006-06-11 Raif S. Naffah <[EMAIL PROTECTED]> + + * gnu/javax/security/auth/login/ConfigFileParser.java (validateClassName): + Check that every component of 'cn' starts with a valid Java identifier char. + 2006-06-10 Tom Tromey <[EMAIL PROTECTED]> * java/io/File.java (pathSeparator): Typo fix. Index: gnu/javax/security/auth/login/ConfigFileParser.java =================================================================== RCS file: /cvsroot/classpath/classpath/gnu/javax/security/auth/login/ConfigFileParser.java,v retrieving revision 1.2 retrieving revision 1.3 diff -u -b -r1.2 -r1.3 --- gnu/javax/security/auth/login/ConfigFileParser.java 9 Jun 2006 16:41:14 -0000 1.2 +++ gnu/javax/security/auth/login/ConfigFileParser.java 11 Jun 2006 00:05:10 -0000 1.3 @@ -268,8 +268,9 @@ for (int i = 0; i < tokens.length; i++) { String t = tokens[i]; - if (Character.isJavaIdentifierStart(cn.toCharArray()[0])) - abort(""); + if (! Character.isJavaIdentifierStart(t.toCharArray()[0])) + abort("Class name [" + cn + + "] contains an invalid sub-package identifier: " + t); // we dont check the rest of the characters for isJavaIdentifierPart() // because that's what the tokenizer does.