On Fri, 5 Jul 2024 14:06:39 GMT, Kevin Walls <kev...@openjdk.org> wrote:

> The test 
> test/jdk/javax/management/remote/mandatory/connection/RMIConnector_NPETest.java
>  should be removed.
> 
> This test was added when 6984520 was fixed in 6u25.  It has been 
> problemlisted since JDK-8267123 removed RMI Activation (it does not use RMI 
> Activation, it just wanted something "RMI" to connect with).
> 
> Looking at the change it tested, the code is no longer the same, and is no 
> longer at risk of this NPE.
> 
> The original NPE fix was to check that jxmServerviceURL was not null before 
> calling methods on it, but the current RMIConnector.java does not make those 
> same calls.
> 
> It is best to remove the test and its problemlist entry.

This pull request has now been integrated.

Changeset: f677b90e
Author:    Kevin Walls <kev...@openjdk.org>
URL:       
https://git.openjdk.org/jdk/commit/f677b90eb93026d3fdfd4ae19d48415a7d8318e8
Stats:     79 lines in 2 files changed: 0 ins; 79 del; 0 mod

8267887: RMIConnector_NPETest.java fails after removal of RMI Activation 
(JDK-8267123)

Reviewed-by: cjplummer, sspitsyn

-------------

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

Reply via email to