RE: [11u] RFR(S): 8223326: Regression introduced by CPU sync: java.security.AccessControlException: access denied ("java.net.NetPermission" "setSocketImpl")
The changeset references JDK-8223326, which is private. If possible, ask Oracle to make it public so we can do a normal backport rather than file an 11u-specific issue. The backport itself looks fine. Thanks, Paul On 3/23/20, 11:08 AM, "jdk-updates-dev on behalf of Doerr, Martin" <jdk-updates-dev-bounces@openjdk.java.net on behalf of martin.doerr@sap.com> wrote: Hi, I'd like to backport JDK-8223326 from jdk/jdk. 11u backport issue: https://bugs.openjdk.java.net/browse/JDK-8241460 Original change: https://hg.openjdk.java.net/jdk/jdk/rev/29624901d8bc 11u backport webrev: http://cr.openjdk.java.net/~mdoerr/8223326_nio_socket_11u/webrev.00/ I had to integrate the change manually due to context changes. However, the new code fits to the 11u versions. Please review. Best regards, Martin
Hi Paul, thanks for the review. We had already asked, but this one can't get opened up. I would have preferred a normal backport, too. Best regards, Martin
-----Original Message----- From: Hohensee, Paul <hohensee@amazon.com> Sent: Montag, 23. März 2020 19:29 To: Doerr, Martin <martin.doerr@sap.com>; core-libs- dev@openjdk.java.net; jdk-updates-dev@openjdk.java.net Subject: RE: [11u] RFR(S): 8223326: Regression introduced by CPU sync: java.security.AccessControlException: access denied ("java.net.NetPermission" "setSocketImpl")
The changeset references JDK-8223326, which is private. If possible, ask Oracle to make it public so we can do a normal backport rather than file an 11u-specific issue. The backport itself looks fine.
Thanks, Paul
On 3/23/20, 11:08 AM, "jdk-updates-dev on behalf of Doerr, Martin" <jdk- updates-dev-bounces@openjdk.java.net on behalf of martin.doerr@sap.com> wrote:
Hi,
I'd like to backport JDK-8223326 from jdk/jdk.
11u backport issue: https://bugs.openjdk.java.net/browse/JDK-8241460
Original change: https://hg.openjdk.java.net/jdk/jdk/rev/29624901d8bc
11u backport webrev:
http://cr.openjdk.java.net/~mdoerr/8223326_nio_socket_11u/webrev.00/
I had to integrate the change manually due to context changes. However, the new code fits to the 11u versions.
Please review.
Best regards, Martin
Hi Paul, as Martin said, the original bug could not be opened by Oracle. I, however changed the bug created by Martin to a Backport item. Doing it that way would allow a "standard" backport of such type of issues while the original item that was pushed to head can remain invisible. We proceeded like that with other bugs already and it proved to work nicely. Best regards Christoph
-----Original Message----- From: core-libs-dev <core-libs-dev-bounces@openjdk.java.net> On Behalf Of Hohensee, Paul Sent: Montag, 23. März 2020 19:29 To: Doerr, Martin <martin.doerr@sap.com>; core-libs- dev@openjdk.java.net; jdk-updates-dev@openjdk.java.net Subject: RE: [11u] RFR(S): 8223326: Regression introduced by CPU sync: java.security.AccessControlException: access denied ("java.net.NetPermission" "setSocketImpl")
The changeset references JDK-8223326, which is private. If possible, ask Oracle to make it public so we can do a normal backport rather than file an 11u-specific issue. The backport itself looks fine.
Thanks, Paul
On 3/23/20, 11:08 AM, "jdk-updates-dev on behalf of Doerr, Martin" <jdk- updates-dev-bounces@openjdk.java.net on behalf of martin.doerr@sap.com> wrote:
Hi,
I'd like to backport JDK-8223326 from jdk/jdk.
11u backport issue: https://bugs.openjdk.java.net/browse/JDK-8241460
Original change: https://hg.openjdk.java.net/jdk/jdk/rev/29624901d8bc
11u backport webrev:
http://cr.openjdk.java.net/~mdoerr/8223326_nio_socket_11u/webrev.00/
I had to integrate the change manually due to context changes. However, the new code fits to the 11u versions.
Please review.
Best regards, Martin
participants (3)
-
Doerr, Martin
-
Hohensee, Paul
-
Langer, Christoph