Continuation of RFR thread from
http://mail.openjdk.java.net/pipermail/security-dev/2020-August/022373.html
CSR has been approved.
-
Commit messages:
- JDK-8250968
Changes: https://git.openjdk.java.net/jdk/pull/56/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=56&ran
> Continuation of RFR thread from
> http://mail.openjdk.java.net/pipermail/security-dev/2020-August/022373.html
>
> CSR has been approved.
Sean Coffey has updated the pull request incrementally with one additional
commit since the last revision:
Copyright and t
On Mon, 7 Sep 2020 13:48:57 GMT, Sean Coffey wrote:
> Continuation of RFR thread from
> http://mail.openjdk.java.net/pipermail/security-dev/2020-August/022373.html
>
> CSR has been approved.
This pull request has now been integrated.
Changeset: 7686e871
Author: Sean
removing the "closing inbound before receiving peer's close_notify" exception
that can be seen with TLS stack if calling close on inbound. After reading the
relevant parts of the TLS v1.2/v1.3 RFCs, I believe the local end point doesn't
have to wait for close_notify alert from remote end.
-
On Thu, 12 Nov 2020 06:44:27 GMT, Matthias Baesken wrote:
>> Marked as reviewed by mbaesken (Reviewer).
>
> Looks good to me !
Meant to comment earlier. Thanks for cleaning this one up!
-
PR: https://git.openjdk.java.net/jdk/pull/1166
On Fri, 13 Nov 2020 21:18:30 GMT, Weijun Wang wrote:
>> Without this method, `PSSParameterSpec::toString` shows something like:
>> MD: SHA-256
>> MGF: java.security.spec.MGF1ParameterSpec@77b52d12
>> SaltLength: 32
>> TrailerField: 1
>> This is ugly.
>>
>> Noreg-trivial.
>
> Do you want me to co
On Mon, 16 Nov 2020 14:25:20 GMT, Weijun Wang wrote:
>> Without this method, `PSSParameterSpec::toString` shows something like:
>> MD: SHA-256
>> MGF: java.security.spec.MGF1ParameterSpec@77b52d12
>> SaltLength: 32
>> TrailerField: 1
>> This is ugly.
>>
>> Noreg-trivial.
>
> Weijun Wang has upda
On Tue, 17 Nov 2020 17:55:19 GMT, Sean Mullan wrote:
>> This change disables the TLSv1 and TLSv1.1 protocols by adding them to the
>> jdk.tls.disabledAlgorithms security property in the java.security file.
>> These protocols use weak algorithms and are being deprecated by the IETF.
>> They sho
Looks fine. Approved for jdk8u-dev.
regards,
Sean.
On 08/07/2016 18:04, Ivan Gerasimov wrote:
Hello!
I'd like to backport this fix into jdk8u-dev.
The fix is essentially the same as in jdk9, but could not be used
verbatim because of the code derivation.
Bug: https://bugs.openjdk.java.net/br
rypto.policy=crypto.policydir-tbd." Does that work for you?
> Instead of throwing an exception here, I wonder if it would make more
> sense to assume a default value of "limited" if the property is not
> set or is empty.
We could, but see above.
Sean Coffey wrote:
&g
On Fri, 13 Nov 2020 14:16:35 GMT, Sean Coffey wrote:
> removing the "closing inbound before receiving peer's close_notify" exception
> that can be seen with TLS stack if calling close on inbound. After reading
> the relevant parts of the TLS v1.2/v1.3 RFCs, I bel
r close_notify alert from remote end.
Sean Coffey has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull request contains three additional commits since
the last revision:
- ve
On Fri, 13 Nov 2020 14:16:35 GMT, Sean Coffey wrote:
> removing the "closing inbound before receiving peer's close_notify" exception
> that can be seen with TLS stack if calling close on inbound. After reading
> the relevant parts of the TLS v1.2/v1.3 RFCs, I bel
On Mon, 11 Jan 2021 04:57:19 GMT, Xue-Lei Andrew Fan wrote:
> The SSLSocketImpl and SSLEngineImpl implementation does not override the
> toString() method. The old code did. BTW, I also made a few code clean up
> in the SSLSocketImpl.java.
>
> Code clean up, trivial update, no new regression
Correction of NPE and updating of test cases. Minor refactoring of test library
also.
-
Commit messages:
- 8255348: NPE in PKIXCertPathValidator event logging code
Changes: https://git.openjdk.java.net/jdk/pull/2150/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2150&
> Correction of NPE and updating of test cases. Minor refactoring of test
> library also.
Sean Coffey has updated the pull request incrementally with one additional
commit since the last revision:
Sean Mullan review update
-
Changes:
- all: https://git.openjdk.java.n
On Tue, 19 Jan 2021 17:54:33 GMT, Sean Coffey wrote:
> Correction of NPE and updating of test cases. Minor refactoring of test
> library also.
This pull request has now been integrated.
Changeset: 18eb6d9e
Author: Sean Coffey
URL: https://git.openjdk.java.net/jdk/commit/18
On Mon, 1 Feb 2021 23:06:30 GMT, Hai-May Chao wrote:
> This change is made for compliance with RFC 5280 section 4.2.1.1 for
> Authority Key Identifier extension.
Marked as reviewed by coffeys (Reviewer).
-
PR: https://git.openjdk.java.net/jdk/pull/2343
On Tue, 9 Feb 2021 12:35:27 GMT, Chris Hegarty wrote:
> This issue adds a new event to improve diagnostic information of Java
> deserialization. The event captures the details of deserialization activity
> from ObjectInputStream. The event details are similar to that of the serial
> filter, bu
On Thu, 11 Feb 2021 15:28:07 GMT, Chris Hegarty wrote:
>> This issue adds a new event to improve diagnostic information of Java
>> deserialization. The event captures the details of deserialization activity
>> from ObjectInputStream. The event details are similar to that of the serial
>> filte
On Fri, 12 Feb 2021 16:26:09 GMT, Chris Hegarty wrote:
>> This issue adds a new event to improve diagnostic information of Java
>> deserialization. The event captures the details of deserialization activity
>> from ObjectInputStream. The event details are similar to that of the serial
>> filte
On Fri, 9 Apr 2021 11:15:40 GMT, Abdul Kolarkunnu
wrote:
>> There is a lack of tests in the area of java.net.Socket.shutdownInput() and
>> java.net.Socket.shutdownOutput() , so added more tests in this area of with
>> different TLS versions. Please review.
>
> Abdul Kolarkunnu has updated the
On Tue, 2 Mar 2021 10:31:03 GMT, Abdul Kolarkunnu
wrote:
>> There is a lack of tests in the area of java.net.Socket.shutdownInput() and
>> java.net.Socket.shutdownOutput() , so added more tests in this area of with
>> different TLS versions. Please review.
>
> @coffeys
Thanks for increasing
Added capability to allow the PKCS11 Token to be destroyed once a session is
logged out from. New configuration properties via pkcs11 config file. Cleaned
up the native resource poller also.
New unit test case to test behaviour. Some PKCS11 tests refactored to allow
pkcs11 provider to be config
Trivial enough change. Improved the exception thrown from JceKeyStore also.
-
Commit messages:
- Fix white space
- 8236671: NullPointerException in JKS keystore
Changes: https://git.openjdk.java.net/jdk/pull/3588/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3588&ran
On Tue, 20 Apr 2021 17:00:40 GMT, Xue-Lei Andrew Fan wrote:
> It looks like a public behavior change to me. Did you want to file a CSR and
> update the specification (KeyStore) as well? I think it would be nice if we
> could keep use the old exception, IllegalArgumentException, as described in
On Mon, 26 Apr 2021 17:29:26 GMT, Sean Mullan wrote:
> This change will restrict JARs signed with SHA-1 algorithms and treat them as
> if they were unsigned. This applies to the algorithms used to digest, sign,
> and optionally timestamp the JAR. It also applies to the signature and digest
> a
> Trivial enough change. Improved the exception thrown from JceKeyStore also.
Sean Coffey has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull request contains f
On Thu, 29 Apr 2021 17:51:17 GMT, Weijun Wang wrote:
>> It's awkward that for a password-less pkcs12 keystore, `keytool -list` does
>> not prompt for a password but `keytool -list -storetype pkcs12` does.
>
> Weijun Wang has updated the pull request incrementally with one additional
> commit si
On Wed, 28 Apr 2021 12:39:42 GMT, Sean Coffey wrote:
>> Trivial enough change. Improved the exception thrown from JceKeyStore also.
>
> Sean Coffey has updated the pull request with a new target base due to a
> merge or a rebase. The incremental webrev excludes the unrelated cha
On Tue, 20 Apr 2021 11:54:39 GMT, Sean Coffey wrote:
> Trivial enough change. Improved the exception thrown from JceKeyStore also.
This pull request has now been integrated.
Changeset: 276a1bf7
Author: Sean Coffey
URL:
https://git.openjdk.java.net/jdk/com
t; pkcs11 provider to be configured (and tested) with a config file of choice.
>
> Reviewer request @valeriepeng
Sean Coffey has updated the pull request incrementally with one additional
commit since the last revision:
Initial corrections from RFR
-
Changes:
- all: http
On Wed, 21 Apr 2021 01:26:18 GMT, Valerie Peng wrote:
>> Added capability to allow the PKCS11 Token to be destroyed once a session is
>> logged out from. New configuration properties via pkcs11 config file.
>> Cleaned up the native resource poller also.
>>
>> New unit test case to test behavio
On Tue, 25 May 2021 18:03:51 GMT, Bradford Wetmore wrote:
> Simple typo fix. Somehow the trailing "u" got omitted, so the code won't
> parse when fed into the compiler.
>
> Resulting javadoc output now compiles.
Looks good!
-
Marked as reviewed by coffeys (Reviewer).
PR: https:
t; pkcs11 provider to be configured (and tested) with a config file of choice.
>
> Reviewer request @valeriepeng
Sean Coffey has updated the pull request with a new target base due to a merge
or a rebase. The pull request now contains five commits:
- whitespace
- Further 8240256 test
On Fri, 16 Apr 2021 11:24:57 GMT, Sean Coffey wrote:
> Added capability to allow the PKCS11 Token to be destroyed once a session is
> logged out from. New configuration properties via pkcs11 config file. Cleaned
> up the native resource poller also.
>
> New unit test case to
MultipleLogins.java should skip test where NSS support is not present
-
Commit messages:
- 8268167: MultipleLogins.java failure on macosx-aarch64
Changes: https://git.openjdk.java.net/jdk/pull/4333/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=4333&range=00
Issue: h
On Thu, 3 Jun 2021 11:10:10 GMT, Sean Coffey wrote:
> MultipleLogins.java should skip test where NSS support is not present
This pull request has now been integrated.
Changeset: eb385c0d
Author: Sean Coffey
URL:
https://git.openjdk.java.net/jdk/com
On Fri, 4 Jun 2021 14:08:58 GMT, Sean Mullan wrote:
>> Hi,
>>
>> Please review my fix for JDK-8255148 which clarifies when an exception
>> contains debug information only.
>>
>> Regards,
>> Evan
>
> src/java.base/share/classes/sun/security/ssl/SSLSocketImpl.java line 590:
>
>> 588:
Sufficient permissions missing if this code was ever to run with
SecurityManager.
Cleanest approach appears to be use of InnocuousThread to create the
cleaner/poller threads.
Test case coverage extended to cover the SecurityManager scenario.
Reviewer request: @valeriepeng
-
Commi
On Tue, 22 Jun 2021 12:01:07 GMT, Sean Coffey wrote:
> Sufficient permissions missing if this code was ever to run with
> SecurityManager.
>
> Cleanest approach appears to be use of InnocuousThread to create the
> cleaner/poller threads.
> Test case coverage ext
On Tue, 22 Jun 2021 12:01:07 GMT, Sean Coffey wrote:
> Sufficient permissions missing if this code was ever to run with
> SecurityManager.
>
> Cleanest approach appears to be use of InnocuousThread to create the
> cleaner/poller threads.
> Test case coverage ext
Sufficient permissions missing if this code was ever to run with
SecurityManager.
Cleanest approach appears to be use of InnocuousThread to create the
cleaner/poller threads.
Test case coverage extended to cover the SecurityManager scenario.
Reviewer request: @valeriepeng
-
Commi
er request: @valeriepeng
Sean Coffey has updated the pull request incrementally with one additional
commit since the last revision:
Move TokenPoller to Runnable
-
Changes:
- all: https://git.openjdk.java.net/jdk17/pull/117/files
- new: https://git.openjdk.java.net/jdk17/pull/117/
er request: @valeriepeng
Sean Coffey has updated the pull request with a new target base due to a merge
or a rebase. The incremental webrev excludes the unrelated changes brought in
by the merge/rebase. The pull request contains four additional commits since
the last revision:
- Edits from review
On Tue, 22 Jun 2021 13:26:41 GMT, Sean Coffey wrote:
> Sufficient permissions missing if this code was ever to run with
> SecurityManager.
>
> Cleanest approach appears to be use of InnocuousThread to create the
> cleaner/poller threads.
> Test case coverage ext
Corner case where a session resumption can fail if the TLS server changes
supported protocol versions in relation to a cached SSLSession. This is
primarily an issue where the legacy TLS version is used in place of the newer
"supported_versions" TLS extension.
-
Commit messages:
-
> Corner case where a session resumption can fail if the TLS server changes
> supported protocol versions in relation to a cached SSLSession. This is
> primarily an issue where the legacy TLS version is used in place of the newer
> "supported_versions" TLS extension.
S
On Wed, 18 Aug 2021 19:03:10 GMT, djelinski
wrote:
>> Corner case where a session resumption can fail if the TLS server changes
>> supported protocol versions in relation to a cached SSLSession. This is
>> primarily an issue where the legacy TLS version is used in place of the
>> newer "suppo
On Wed, 18 Aug 2021 22:33:42 GMT, Weijun Wang wrote:
> The "Key for the principal foo...@acme.com not available in
> /home/foobar/foobar.keytab" debug output does not contain enough information.
> The keytab file might be missing, or not readable, or does not contain the
> required key(s).
>
> Corner case where a session resumption can fail if the TLS server changes
> supported protocol versions in relation to a cached SSLSession. This is
> primarily an issue where the legacy TLS version is used in place of the newer
> "supported_versions" TLS extension.
S
> Corner case where a session resumption can fail if the TLS server changes
> supported protocol versions in relation to a cached SSLSession. This is
> primarily an issue where the legacy TLS version is used in place of the newer
> "supported_versions" TLS extension.
S
> Corner case where a session resumption can fail if the TLS server changes
> supported protocol versions in relation to a cached SSLSession. This is
> primarily an issue where the legacy TLS version is used in place of the newer
> "supported_versions" TLS extension.
S
On Thu, 19 Aug 2021 19:48:15 GMT, Xue-Lei Andrew Fan wrote:
>> Sean Coffey has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Remove redundant method and testcase cleanup
>
> src/java.base/share/classes/sun
On Thu, 19 Aug 2021 19:51:36 GMT, Xue-Lei Andrew Fan wrote:
>> Sean Coffey has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> maxProtocolVersion refactoring
>
> test/jdk/sun/security/ssl/SSLSessionImpl/
On Fri, 13 Aug 2021 14:00:45 GMT, Sean Coffey wrote:
> Corner case where a session resumption can fail if the TLS server changes
> supported protocol versions in relation to a cached SSLSession. This is
> primarily an issue where the legacy TLS version is used in place of
Use correct manifest file name in the Manifest verifier checks.
Also - extra null check
The test doesn't reproduce the exact issue reported but should prevent future
regressions in this area.
-
Commit messages:
- 8273826: Correct Manifest file name and NPE checks
Changes: https:/
> Use correct manifest file name in the Manifest verifier checks.
> Also - extra null check
>
> The test doesn't reproduce the exact issue reported but should prevent future
> regressions in this area.
Sean Coffey has updated the pull request incrementally with one addition
On Wed, 6 Oct 2021 17:54:15 GMT, Weijun Wang wrote:
>> Sean Coffey has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Make variables final
>
> src/java.base/share/classes/sun/security/util/ManifestEntryVeri
On Thu, 7 Oct 2021 14:57:53 GMT, Sean Mullan wrote:
>> Sean Coffey has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Make variables final
>
> test/jdk/sun/security/tools/jarsigner/warnings/LowerCaseM
On Wed, 6 Oct 2021 16:58:51 GMT, Sean Coffey wrote:
> Use correct manifest file name in the Manifest verifier checks.
> Also - extra null check
>
> The test doesn't reproduce the exact issue reported but should prevent future
> regressions in this area.
This pull
Some elements of the PKCS9Attribute.PKCS9_OIDS array may have null value. The
PKCS9Attributes.toString() and PKCS9Attributes.getAttributes() methods need to
account for that.
-
Commit messages:
- 8277224: sun.security.pkcs.PKCS9Attributes.toString() throws NPE
Changes: https://git
On Wed, 17 Nov 2021 16:52:46 GMT, Weijun Wang wrote:
>> Some elements of the PKCS9Attribute.PKCS9_OIDS array may have null value.
>> The PKCS9Attributes.toString() and PKCS9Attributes.getAttributes() methods
>> need to account for that.
>
> test/jdk/sun/security/x509/AlgorithmId/NonStandardName
> Some elements of the PKCS9Attribute.PKCS9_OIDS array may have null value. The
> PKCS9Attributes.toString() and PKCS9Attributes.getAttributes() methods need
> to account for that.
Sean Coffey has updated the pull request incrementally with one additional
commit since the last
On Wed, 17 Nov 2021 16:00:04 GMT, Sean Coffey wrote:
> Some elements of the PKCS9Attribute.PKCS9_OIDS array may have null value. The
> PKCS9Attributes.toString() and PKCS9Attributes.getAttributes() methods need
> to account for that.
This pull request has now been integrated.
On Wed, 12 Jan 2022 21:57:22 GMT, Sean Mullan wrote:
> If a JAR is signed with multiple digest algorithms and one of the digest
> algorithms is disabled, `ManifestEntryVerifier.verify()` was incorrectly
> returning null indicating that the jar entry has no signers.
>
> This fixes the issue su
On Wed, 12 Jan 2022 21:57:22 GMT, Sean Mullan wrote:
> If a JAR is signed with multiple digest algorithms and one of the digest
> algorithms is disabled, `ManifestEntryVerifier.verify()` was incorrectly
> returning null indicating that the jar entry has no signers.
>
> This fixes the issue su
On Thu, 13 Jan 2022 13:56:14 GMT, Sean Mullan wrote:
>> src/java.base/share/classes/sun/security/util/ManifestEntryVerifier.java
>> line 212:
>>
>>> 210:
>>> 211: CodeSigner[] entrySigners = sigFileSigners.get(name);
>>> 212: Map permittedAlgs =
>>
>> maybe permittedAlgsChecke
On Sat, 12 Mar 2022 00:55:07 GMT, Bradford Wetmore wrote:
> JDK-8253368 changed the behavior of SSLSocket to no longer throw a fatal
> internal_error (80) and invalidate existing sessions (either completed or
> under construction) as described in (RFC 4346/TLSv1.1+) if a connection was
> close
On Tue, 22 Mar 2022 00:24:41 GMT, Bradford Wetmore wrote:
>> test/jdk/sun/security/ssl/SSLSocketImpl/SSLSocketSSLEngineCloseInbound.java
>> line 130:
>>
>>> 128: * The following is to set up the keystores/trust material.
>>> 129: */
>>> 130: private static final String pathToStore
On Sat, 12 Mar 2022 00:55:07 GMT, Bradford Wetmore wrote:
> JDK-8253368 changed the behavior of SSLSocket to no longer throw a fatal
> internal_error (80) and invalidate existing sessions (either completed or
> under construction) as described in (RFC 4346/TLSv1.1+) if a connection was
> close
On Fri, 1 Apr 2022 07:32:21 GMT, Andrey Turbanov wrote:
> Let's take advantage of Java 7 language feature - "Catching Multiple
> Exception Types".
> It simplifies code. Reduces duplication.
> Found by IntelliJ IDEA inspection `Identical 'catch' branches in 'try'
> statement`
Looks fine!
-
On Tue, 12 Apr 2022 11:28:12 GMT, Daniel Jeliński wrote:
> During TLS handshake, hundreds of constraints are evaluated to determine
> which cipher suites are usable. Most of the evaluations are performed using
> `HandshakeContext#algorithmConstraints` object. By default that object
> contains
On Thu, 21 Apr 2022 19:58:39 GMT, Daniel Jeliński wrote:
> Profiling the TLS handshakes using SSLHandshake benchmark shows that a large
> portion of time is spent in HandshakeContext initialization, specifically in
> DisabledAlgorithmConstraints class.
>
> There are only a few instances of tha
Changeset: 00c9d6cce3ec
Author:coffeys
Date: 2012-06-15 14:16 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/00c9d6cce3ec
7156963: Incorrect copyright header in java/io/SerialCallbackContext
Reviewed-by: weijun, coffeys
Contributed-by: edvard.wende...@oracle.com
! src/share/
Changeset: 612e56cf284c
Author:coffeys
Date: 2012-06-27 21:10 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/612e56cf284c
6893617: JDK 6 CNCtx always uses the default ORB
Reviewed-by: lancea
! src/share/classes/com/sun/jndi/cosnaming/CNCtx.java
Changeset: 47adb42076f1
Author:coffeys
Date: 2012-06-27 21:09 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/47adb42076f1
7162902: Umbrella port of a number of corba bug fixes from JDK 6 to jdk7u/8
Reviewed-by: lancea
! src/share/classes/com/sun/corba/se/impl/encoding/Cach
Changeset: 35fec642fd32
Author:coffeys
Date: 2012-07-26 22:00 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/35fec642fd32
7179879: SSLSocket connect times out instead of throwing socket closed exception
Reviewed-by: xuelei, chegar
! src/share/classes/sun/security/ssl/SSLSock
Changeset: 39b01268b845
Author:coffeys
Date: 2012-08-16 10:35 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/39b01268b845
7056731: Race condition in CORBA code causes re-use of ABORTed connections
Reviewed-by: lancea
Contributed-by: d.macdon...@auckland.ac.nz
! test/Makefile
Changeset: 18a02ad8dc73
Author:coffeys
Date: 2012-08-16 10:33 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/18a02ad8dc73
7056731: Race condition in CORBA code causes re-use of ABORTed connections
Reviewed-by: lancea
Contributed-by: d.macdon...@auckland.ac.nz
!
src/share/
Changeset: 56d8756749bd
Author:coffeys
Date: 2012-08-16 10:48 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/56d8756749bd
7185965: Build error in javadoc make stage for bundles not containing crypto
package
Reviewed-by: chegar
! make/common/shared/Defs-java.gmk
Changeset: da1436b21bc2
Author:coffeys
Date: 2012-08-31 12:25 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/da1436b21bc2
7195063: [TEST] jtreg flags com/sun/corba/cachedSocket/7056731.sh with Error
failure.
Reviewed-by: chegar
! test/com/sun/corba/cachedSocket/7056731.sh
Changeset: fffbb33df102
Author:coffeys
Date: 2012-09-07 21:22 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/fffbb33df102
7180362: RFE: Implement date cutover functionality for currency.properties file
Reviewed-by: naoto
! src/share/classes/java/util/Currency.java
! test/jav
Changeset: fecba6a8b78e
Author:coffeys
Date: 2012-10-09 12:50 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/fecba6a8b78e
7196533: TimeZone.getDefault() slow due to synchronization bottleneck
Reviewed-by: okutsu
! src/share/classes/java/util/TimeZone.java
Changeset: 71de5e31d497
Author:coffeys
Date: 2012-10-09 19:45 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/71de5e31d497
7181793: Socket getOutputStream create streams that cannot be GC'ed until
Socket is closed
Reviewed-by: alanb, chegar
! src/share/classes/java/net/Abstr
Changeset: 679e8ad9874f
Author:coffeys
Date: 2012-10-09 20:14 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/679e8ad9874f
7196086: update copyright years for files in corba repository (JDK 8)
Reviewed-by: lancea
! make/common/Defs-bsd.gmk
! make/common/internal/Resources.g
Changeset: 885bb24f6018
Author:coffeys
Date: 2013-02-19 14:07 +
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/885bb24f6018
7197187: Currency.isPastCutoverDate should be made more robust
Reviewed-by: alanb
! src/share/classes/java/util/Currency.java
Changeset: 01b6b0dd2006
Au
Changeset: 5871d7b1673c
Author:coffeys
Date: 2013-04-25 21:12 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5871d7b1673c
8000529: Regression: SimpleDateFormat incorrectly parses dates formatted with Z
and z pattern letters
Reviewed-by: okutsu
! src/share/classes/java/text/
Changeset: c4480e0d9f53
Author:coffeys
Date: 2013-06-06 14:10 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c4480e0d9f53
8000450: Restrict access to com/sun/corba/se/impl package
Reviewed-by: alanb, chegar, lancea
! src/share/lib/security/java.security-linux
! src/share/lib
Changeset: 45a3584bfacf
Author:coffeys
Date: 2013-06-14 15:14 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/45a3584bfacf
8015978: Incorrect transformation of XPath expression "string(-0)"
Reviewed-by: darcy, joehw
Contributed-by: aleksej.efi...@oracle.com
+ test/javax/xml/j
Changeset: 659828443145
Author:coffeys
Date: 2013-06-14 15:14 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/659828443145
8015978: Incorrect transformation of XPath expression "string(-0)"
Reviewed-by: darcy, joehw
Contributed-by: aleksej.efi...@oracle.com
! src/com/sun/org
Changeset: 83c2976ef8ee
Author:coffeys
Date: 2013-07-09 16:00 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/83c2976ef8ee
8019979: Replace CheckPackageAccess test with better one from closed repo
Reviewed-by: mullan
! test/java/lang/SecurityManager/CheckPackageAccess.java
Changeset: 7cc35dd1885d
Author:coffeys
Date: 2013-07-15 13:42 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/7cc35dd1885d
8017566: Backout 8000450 - Cannot access to com.sun.corba.se.impl.orb.ORBImpl
Reviewed-by: mchung
! src/share/lib/security/java.security-linux
! src/shar
Changeset: 92a66af7f834
Author:igerasim
Date: 2013-08-26 18:26 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/92a66af7f834
8016018: Typo in AbstractStringBuilder#indexOf and #lastIndexOf descriptions
Reviewed-by: alanb, chegar
! src/share/classes/java/lang/AbstractStringBuil
Changeset: 5920155dd080
Author:coffeys
Date: 2013-09-03 22:37 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5920155dd080
8017195: Introduce option to setKeepAlive parameter on CORBA sockets
Reviewed-by: chegar, msheppar
+ test/com/sun/corba/transport/KeepAliveSockets.java
Changeset: af8e5bc3a150
Author:coffeys
Date: 2013-09-03 22:35 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/af8e5bc3a150
8017195: Introduce option to setKeepAlive parameter on CORBA sockets
Reviewed-by: chegar, msheppar
!
src/share/classes/com/sun/corba/se/impl/transport
Changeset: 368172cb6dc5
Author:coffeys
Date: 2013-10-02 09:21 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/368172cb6dc5
8024952: ClassCastException in PlainSocketImpl.accept() when using custom
socketImpl
Reviewed-by: chegar
! src/windows/classes/java/net/PlainSocketImpl.
Changeset: 60e3cdbe8cdf
Author:aefimov
Date: 2013-10-13 14:19 +0400
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/60e3cdbe8cdf
8025255: (tz) Support tzdata2013g
Reviewed-by: okutsu, mfang
! make/sun/javazic/tzdata/VERSION
! make/sun/javazic/tzdata/africa
! make/sun/javazic/tzdata
Changeset: 2af3f5a61322
Author:coffeys
Date: 2013-10-23 16:53 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/2af3f5a61322
5036554: unmarshal error on CORBA alias type in CORBA any
Reviewed-by: chegar, smarks
+ test/com/sun/corba/5036554/JavaBug.java
+ test/com/sun/corba/5036
Changeset: a90e9efa4264
Author:coffeys
Date: 2013-10-23 16:45 +0100
URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/a90e9efa4264
5036554: unmarshal error on CORBA alias type in CORBA any
Reviewed-by: chegar, smarks
! src/share/classes/com/sun/corba/se/impl/corba/AnyImpl.java
1 - 100 of 137 matches
Mail list logo