On Sat, 12 Aug 2023 03:56:05 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:

> VMConnection.open() expects launchTarget(), attachTarget(), and 
> listentTarget() to either throw an exception or return a valid VirtualMachine 
> instance. Instead they were catching certain exceptions and returning null, 
> which resulted in an NPE in VMConnection.open(). I've fixed it so these APIs 
> now rethrow any caught exception and never return null.
> 
> Tested with tier1 and also running the following 10x each on linux-x64, 
> macosx-aarch64, and windows-x64:
> 
> test/hotspot/jtreg/vmTestbase/nsk/jdwp/
> test/hotspot/jtreg/vmTestbase/nsk/jdb/
> test/hotspot/jtreg/vmTestbase/nsk/jdi
> test/jdk/com/sun/jdi

Ping! I could use one more review please. Thanks

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

PR Comment: https://git.openjdk.org/jdk/pull/15255#issuecomment-1681036548

Reply via email to