Re: CFV: New JDK7u Committer: Anton Nashatyrev

2014-08-05 Thread Sean Mullan
Vote: yes --Sean On 08/05/2014 10:06 AM, Alexander Potochkin wrote: I hereby nominate Anton Nashatyrev to jdk7u Committer. Anton is a member of the Java SE sustaining team. He has spent most of that time working on the GUI and deployment issues. He has contributed 16 changes to jdk7u so far:

Re: [7u40] Request for phase 2 review and approval for 8019979

2013-07-11 Thread Sean Mullan
Looks good. --Sean On 07/11/2013 09:28 AM, Seán Coffey wrote: 8019979 : Replace CheckPackageAccess test with better one from closed repo I'd like to push this testcase enhancement to 7u40. It was pushed to jdk8 TL repo earlier in the week. I've run tests via JPRT and saw no issues. The packag

hg: jdk7u/jdk7u40-dev/jdk: 8017173: XMLCipher with RSA_OAEP Key Transport algorithm can't be instantiated

2013-07-11 Thread sean . mullan
Changeset: c5d869453212 Author:mullan Date: 2013-07-11 13:13 -0400 URL: http://hg.openjdk.java.net/jdk7u/jdk7u40-dev/jdk/rev/c5d869453212 8017173: XMLCipher with RSA_OAEP Key Transport algorithm can't be instantiated Reviewed-by: xuelei, vinnie ! src/share/classes/com/sun/org/apac

[7u40] Request for phase 2 approval for CR 8017173 - XMLCipher with RSA_OAEP Key Transport algorithm can't be instantiated

2013-07-11 Thread Sean Mullan
Bug: http://bugs.sun.com/view_bug.do?bug_id=8017173 Webrev: http://cr.openjdk.java.net/~mullan/webrevs/8017173/webrev.00/ Review Thread: http://mail.openjdk.java.net/pipermail/security-dev/2013-July/008112.html This bug fix is needed because it is a serious regression introduced in 7u25 and

Re: [7u40] Resolve ambiguity in OCSPChecker & CrlRevocationChecker

2013-05-23 Thread Sean Mullan
On 05/22/2013 02:25 PM, Andrew Hughes wrote: Webrev: http://cr.openjdk.java.net/~andrew/7u/webrev/ Looks fine to me. --Sean There's an ambiguity that applies when building OpenJDK 7 with OpenJDK 6 (and hence I haven't posted this for 8 as it can't be built with 6). OpenJDK 6 has a class s

[7u40] Request for approval for CR 7199143 - RFE: OCSP revocation checker should provide possibility to specify connection timeout

2013-05-08 Thread Sean Mullan
This is a request for a retroactive approval for the backport of the following bug fix which was accidentally pushed to jdk7u-dev. * Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7199143 * JDK 8 changeset: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/ee9846f351d7 Thanks, Sean

[7u40] Request for approval for CR 8011313 - OCSP timeout set to wrong value if com.sun.security.ocsp.timeout not defined

2013-05-08 Thread Sean Mullan
This is a request for a retroactive approval for the backport of the following bug fix which was accidentally pushed to jdk7u-dev. * Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8011313 * JDK 8 changeset: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/78d08fc2dd12 Thanks, Sean

[7u40] Request for approval for CR 8013228 - Create new system properties to control allowable OCSP clock skew and CRL connection timeout

2013-05-08 Thread Sean Mullan
This is a request for a retroactive approval for the backport of the following bug fix which was accidentally pushed to jdk7u-dev. * Bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8013228 * JDK 8 changeset: http://hg.openjdk.java.net/jdk8/jdk8/jdk/rev/3e282678a885 Thanks, Sean

Re: CFV: New JDK 7 Updates Project Committer: Edvard Wendelin

2013-02-22 Thread Sean Mullan
Vote: yes On 02/22/2013 09:35 AM, Dalibor Topic wrote: I hereby nominate Edvard Wendelin to JDK 7 Updates Project Committer. As one of this Project's Maintainers, Edvard has been active in this Project since its creation. He has gradually accumulated the necessary commits by contributing across

hg: jdk7u/jdk7u-dev/jdk: 8002313: TEST_BUG : jdk/test/java/security/Security/ClassLoaderDeadlock/ClassLoaderDeadlock.java should run in headless mode

2013-01-31 Thread sean . mullan
Changeset: b213a76d7189 Author:juh Date: 2013-01-31 16:47 -0500 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/b213a76d7189 8002313: TEST_BUG : jdk/test/java/security/Security/ClassLoaderDeadlock/ClassLoaderDeadlock.java should run in headless mode Reviewed-by: mullan !

hg: jdk7u/jdk7u-dev/jdk: 8004188: Rename src/share/lib/security/java.security to java.security-linux

2012-12-11 Thread sean . mullan
Changeset: c6d441f47bae Author:juh Date: 2012-12-11 14:33 -0500 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/c6d441f47bae 8004188: Rename src/share/lib/security/java.security to java.security-linux Reviewed-by: mullan, mchung ! make/java/security/Makefile - src/share/lib

Re: 7168191: Signature validation can fail under certain circumstances

2012-06-18 Thread Sean Mullan
Hi Vinnie, Here are my comments. * OCSPRequest [119-20] Should use debug.println instead of System.out.println * OCSPChecker [386] why do you decrement remainingCerts? Is it identifying the wrong certificate for some reason? [473] Is this called by any other class? If not, I suggest making it

hg: jdk7u/jdk7u4-dev/jdk: 2 new changesets

2012-03-07 Thread sean . mullan
Changeset: e84d607e9b13 Author:mullan Date: 2012-03-07 09:16 -0500 URL: http://hg.openjdk.java.net/jdk7u/jdk7u4-dev/jdk/rev/e84d607e9b13 7146431: java.security files out-of-sync Reviewed-by: vinnie, xuelei, wetmore ! src/share/lib/security/java.security-macosx ! src/share/lib/secur

[7u4] Request for approval for CR 7146431: java.security files out-of-sync

2012-03-06 Thread Sean Mullan
This is a fix to correct an issue in 7u4 with the system-specific java.security files being out-of-sync. bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7146431 webrev: http://cr.openjdk.java.net/~mullan/webrevs/7146431/webrev.00/ The fix does not affect JDK 8. Reviewed by Vincent Ryan a

Re: CR 7149144 - rt.jar: missing sun.security.x509.X500Signer class

2012-02-28 Thread Sean Mullan
On 2/28/12 11:21 AM, Chris Hegarty wrote: > [cc'ing security-dev@o.j.n, they may be able to suggest a suitable > alternative, supported, API to use]. I would recommend they implement the functionality entirely in the OpenDS product. X500Signer is a very simple class and can be implemented using t

hg: jdk7u/jdk7u-dev/jdk: 2 new changesets

2012-01-24 Thread sean . mullan
Changeset: 6d17a729259d Author:mullan Date: 2012-01-24 11:27 -0500 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/6d17a729259d 7131084: XMLDSig XPathFilter2Transform regression involving intersect filter Reviewed-by: xuelei ! src/share/classes/com/sun/org/apache/xml/inter

[7u4] Request for approval for CR 7131084: XMLDSig XPathFilter2Transform regression involving intersect filter

2012-01-24 Thread Sean Mullan
This is a fix for a regression introduced in JDK 7. bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7131084 Same patch as JDK 8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3df0bd3ed880 Reviewed by Xuelei (Andrew) Fan: http://mail.openjdk.java.net/pipermail/security-dev/2012-Jan

hg: jdk7u/jdk7u-dev/jdk: 2 new changesets

2012-01-13 Thread sean . mullan
Changeset: 7617df64c231 Author:mullan Date: 2012-01-13 09:47 -0500 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/7617df64c231 7094155: JSR105 code throws javax.xml.crypto.URIReferenceException when running into Java 7 VM Reviewed-by: xuelei ! src/share/classes/com/sun/o

[7u4] Request for approval for CR 7094155 JSR105 code throws javax.xml.crypto.URIReferenceException when running into Java 7 VM

2012-01-13 Thread Sean Mullan
(Resending; fixed link to bug) bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7094155 Same patch as JDK 8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/56cc907fc8dc Integrated in b14 of JDK 8. --Sean

[7u4] Request for approval for CR 7094155 JSR105 code throws javax.xml.crypto.URIReferenceException when running into Java 7 VM

2012-01-13 Thread Sean Mullan
bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7093090 Same patch as JDK 8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/56cc907fc8dc Integrated in b14 of JDK 8. --Sean

hg: jdk7u/jdk7u-dev/jdk: 7093090: Reduce synchronization in java.security.Policy.getPolicyNoCheck

2011-11-30 Thread sean . mullan
Changeset: fab771990eb3 Author:mullan Date: 2011-11-30 14:48 -0500 URL: http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/fab771990eb3 7093090: Reduce synchronization in java.security.Policy.getPolicyNoCheck Reviewed-by: valeriep ! src/share/classes/java/security/Policy.java

[7u4] Request for approval for CR 7093090 - Reduce synchronization in java.security.Policy.getPolicyNoCheck

2011-11-30 Thread Sean Mullan
bug: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7093090 Same patch as JDK 8 changeset: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/1945abeb82a0 Thanks, Sean

Re: Interaction with the CCC Process - Draft

2011-09-01 Thread Sean Mullan
For non-Oracle developers, will there be a CCC template that developers can use to write the details of the CCC and hand it off to the technical lead? How do we manage comments that come up during the review of the CCC? Is the tech lead expected to be a middle-man and send them back and forth to t