Re: [cp-patches] Patch: X500Principal fixlet

2006-04-06 Thread Tom Tromey
> "Nektarios" == Nektarios K Papadopoulos <[EMAIL PROTECTED]> writes:

Nektarios> A small fix for the readAttributeValue

Thanks for the patch.  I checked it in.

Tom



[cp-patches] Patch: X500Principal fixlet

2006-03-29 Thread Nektarios K. Papadopoulos

A small fix for the readAttributeValue

2006-03-29 Nektarios K. Papadopoulos 

* javax/security/auth/x500/X500Principal.java
(readAttributeValue) Check for separator after quoted value was
failing in all cases.
Index: javax/security/auth/x500/X500Principal.java
===
RCS file: 
/cvsroot/classpath/classpath/javax/security/auth/x500/X500Principal.java,v
retrieving revision 1.11
diff -u -r1.11 X500Principal.java
--- javax/security/auth/x500/X500Principal.java 1 Oct 2005 18:48:46 -   
1.11
+++ javax/security/auth/x500/X500Principal.java 29 Mar 2006 16:03:13 -
@@ -395,7 +395,7 @@
   buf.append((char) ch);
   }
 sep = in.read();
-if (sep != '+' || sep != ',')
+if (sep != '+' && sep != ',')
   throw new IOException("illegal character: " + (char) ch);
 return buf.toString();
   }