[jdk23] Integrated: 8333854: IllegalAccessError with proxies after JDK-8332457

2024-06-21 Thread Chen Liang
On Thu, 20 Jun 2024 20:11:06 GMT, Chen Liang  wrote:

> Please review this patch, which is a backport of the fix in #19615 to JDK 23.
> 
> This is not a clean patch, because the old patch was done on JDK-8333479 
> (#19585) which was absent in JDK 23; however, the conflicts were small, and 
> the only real changes were that `methodTypeDesc` and `classDesc` from 
> `ConstantUtils` were not available, so the original approaches were retained. 
> There is also import adjustments.
> 
> Testing: tier 1-3

This pull request has now been integrated.

Changeset: 1dbad805
Author:Chen Liang 
URL:   
https://git.openjdk.org/jdk/commit/1dbad8058b79c4e31e389913a7259323a9b8ab93
Stats: 238 lines in 2 files changed: 161 ins; 9 del; 68 mod

8333854: IllegalAccessError with proxies after JDK-8332457

Reviewed-by: asotona
Backport-of: 91bd85d65dff9cea91b88da7ef241be5c7b85f94

-

PR: https://git.openjdk.org/jdk/pull/19815


Integrated: 8333854: IllegalAccessError with proxies after JDK-8332457

2024-06-18 Thread Chen Liang
On Mon, 10 Jun 2024 01:32:09 GMT, Chen Liang  wrote:

> Please review this patch that fixes a critical issue that breaks some Proxy 
> usages.
> 
> Since the problematic patch from before cannot be backed out, this patch aims 
> to emulate the old behavior before. A diff between before the problematic 
> patch and this patch is available at 
> https://gist.github.com/7565b2091008f561eb0ada019bc5e517, generated by 
> running `git diff 326dbb1b139dd1ec1b8605339b91697cdf49da9a -- 
> src/java.base/share/classes/java/lang/reflect/ProxyGenerator.java`.

This pull request has now been integrated.

Changeset: 91bd85d6
Author:Chen Liang 
URL:   
https://git.openjdk.org/jdk/commit/91bd85d65dff9cea91b88da7ef241be5c7b85f94
Stats: 238 lines in 2 files changed: 161 ins; 8 del; 69 mod

8333854: IllegalAccessError with proxies after JDK-8332457

Reviewed-by: redestad, asotona

-

PR: https://git.openjdk.org/jdk/pull/19615