code review request: 7184815 (was Re: OpenJDK krb5 ignore /etc/krb5.conf?)

2012-07-17 Thread Weijun Wang
7184815: [macosx] Need to read Kerberos config in files Please take a review: http://cr.openjdk.java.net/~weijun/7184815/webrev.00/ I break the config setting to Java setting and native setting, and insert the reading of SCDynamicStoreConfig between the two. This should preserve the 6u beh

Re: OpenJDK krb5 ignore /etc/krb5.conf?

2012-07-17 Thread Mike Swingler
On Jul 16, 2012, at 8:32 PM, Weijun Wang wrote: > Ping again. > > On 07/05/2012 04:34 PM, Weijun Wang wrote: >> Hi Scott >> >> On Mac since Lion, sun.security.krb5.Config tries to locate the config >> info in this order: >> >> 1. java.security.krb5.conf system property >> 2. ${jre}/lib/securit

hg: jdk8/tl/jdk: 7183053: Optimize DoubleByte charset for String.getBytes()/new String(byte[])

2012-07-17 Thread xueming . shen
Changeset: c76ad79a5a2f Author:sherman Date: 2012-07-17 19:57 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c76ad79a5a2f 7183053: Optimize DoubleByte charset for String.getBytes()/new String(byte[]) Summary: DoubleByte implements sun/nio.cs/ArrayDe/Encoder interface Reviewed

Re: OpenJDK krb5 ignore /etc/krb5.conf?

2012-07-17 Thread Weijun Wang
I'm not familiar with how Mac does it, but normally there are two ways a Kerberos authentication is performed, through the initial login and through kinit. The former is integrated into the system (a pam module?) and I guess in this case the config is inside SCDynamicStoreConfig. For the latter

hg: jdk8/tl/jdk: 7142596: RMI JPRT tests are failing

2012-07-17 Thread stuart . marks
Changeset: b6f78869c66d Author:dmocek Date: 2012-07-17 11:01 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b6f78869c66d 7142596: RMI JPRT tests are failing Summary: Changed RMI tests to use random port numbers for the RMI Registry and RMID so the tests can be run concurrent

Re: Code review request: 7102106: TEST_BUG: sun/security/util/Oid/S11N.sh should be modified

2012-07-17 Thread Sean Mullan
On 7/16/12 9:36 PM, Weijun Wang wrote: > Also, the test skips an old JDK (and shows a warning) if its path does > not exist. What this bug wants to solve is that the path exists but is > too slow. Ok, got it now. --Sean > > -Max > > > On 07/16/2012 10:43 PM, Wei-jun Wang wrote: >> The interop

Re: Request for Review: AESCrypt.init performance JDK change

2012-07-17 Thread Frances Ho
Valerie Peng will be looking into this one. _Frances On 7/17/2012 8:31 AM, Alan Bateman wrote: On 17/07/2012 16:10, Deneau, Tom wrote: Alan -- Can you tell me the procedure I should follow? I have submitted hotspot webrevs before but not JDK webrevs? -- Tom You'll need to get a sponsor

Re: Request for Review: AESCrypt.init performance JDK change

2012-07-17 Thread Alan Bateman
On 17/07/2012 16:10, Deneau, Tom wrote: Alan -- Can you tell me the procedure I should follow? I have submitted hotspot webrevs before but not JDK webrevs? -- Tom You'll need to get a sponsor from the security area (you've cc'ed security-dev so that's the right list). The initial target shou

RE: Request for Review: AESCrypt.init performance JDK change

2012-07-17 Thread Deneau, Tom
Alan -- Can you tell me the procedure I should follow? I have submitted hotspot webrevs before but not JDK webrevs? -- Tom -Original Message- From: Alan Bateman [mailto:[email protected]] Sent: Tuesday, July 17, 2012 3:00 AM To: Deneau, Tom Cc: [email protected]; security

Re: Request for Review: AESCrypt.init performance JDK change

2012-07-17 Thread Alan Bateman
On 16/07/2012 21:08, Deneau, Tom wrote: Please review the following webrev which improves performance for the com/sun/crypto/provider when the algorithm is AES for some kinds of Cipher usage patterns. http://cr.openjdk.java.net/~tdeneau/aescrypt-init-performance/webrev.01/ I do not have a bug n