Integrated: 8355453: nsk.share.jdi.Debugee.waitingEvent() does not timeout properly
Chris Plummer
cjplummer at openjdk.org
Wed Apr 30 17:12:52 UTC 2025
On Thu, 24 Apr 2025 05:28:09 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:
> nsk.share.jdi.Debugee.waitingEvent() takes a timeout argument. However, if the call to EventQueue.remove() times out and returns null, waitingEvent() just keep on retrying. The logic is incorrect for a null result. It should immediately return null. The retry path is only meant for filtering out other events, and it updates "timeLeft' before retrying to avoid endlessly repeating the loop.
>
> Tested by running all of nsk/jdi locally on linux-x64-debug
This pull request has now been integrated.
Changeset: 486acc06
Author: Chris Plummer <cjplummer at openjdk.org>
URL: https://git.openjdk.org/jdk/commit/486acc06e0325d247a96df8f7fc88c9111c3315d
Stats: 4 lines in 1 file changed: 0 ins; 1 del; 3 mod
8355453: nsk.share.jdi.Debugee.waitingEvent() does not timeout properly
Reviewed-by: lmesnik, amenkov, sspitsyn
-------------
PR: https://git.openjdk.org/jdk/pull/24839
More information about the serviceability-dev
mailing list