Code review request: 7059542 JNDI name operations should be locale independent (original) (raw)
Xuelei Fan xuelei.fan at oracle.com
Sun Aug 28 03:36:39 PDT 2011
- Previous message: hg: jdk8/tl/jdk: 7060790: (fs) FileSystem.newWatchService error message confusing when maximum inotify descriptors reached
- Next message: Code review request: 7059542 JNDI name operations should be locale independent
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Hi Weijun,
Would you please review the new update? I also include the changes in security components.
webrev: webrev: http://cr.openjdk.java.net/~xuelei/7059542/webrev.01/
Thanks, Xuelei
On 8/1/2011 9:48 AM, Xuelei Fan wrote:
Hi Weijun,
Please pending the code review. Another thread is addressing the warning issues in JNDI, I will wait for a while to remove the warning update from this fix. Thanks, Xuelei On 7/30/2011 9:52 PM, Xuelei Fan wrote: Hi Weijun,
It was "test/javax/naming/ldap/LdapName/CompareToEqualsTests fails intermittently". I updated the synopsis to "JNDI name operations should be locale independent" when found the underlying failure problem. Would you please review the JNDI update when you available? The fix also include Alan's fix of JNDI compiler warnings. Thanks, Xuelei On 7/25/2011 8:39 AM, Xuelei Fan wrote: Ping ...
Xuelei On 7/21/2011 3:26 PM, Xuelei Fan wrote: Hi,
In JNDI implementation, String.toUpperCase() and String.toLowerCase() is used to compare or hashcode case-insensitive strings. These operations are locale dependent, and may result in unexpected behaviors in some locale.[1] This fix is try to interpret case-insensitive string locale independently in JNDI component. webrev: http://cr.openjdk.java.net/~xuelei/7059542/webrev.00/ Thanks, Xuelei [1]: http://sim.ivi.co/2011/07/trap-of-case-insensitive-string.html
- Previous message: hg: jdk8/tl/jdk: 7060790: (fs) FileSystem.newWatchService error message confusing when maximum inotify descriptors reached
- Next message: Code review request: 7059542 JNDI name operations should be locale independent
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]