[jdk19] Integrated: 8288854: getLocalGraphicsEnvironment() on for multi-screen setups throws exception NPE

Sergey Bylokhov serb at openjdk.org
Sat Jul 2 00:27:45 UTC 2022


On Mon, 27 Jun 2022 21:17:54 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

> This is the fix for a copy-paste error. The fix JDK-8076313 replaced the usage of the "screens" array from the parent class to the "devices" where the list of devices is now maintained. Since "screens" array is never used nor initialized its usage caused an NPE. That check was copied as-is, while it should use the actual number of screen devices requested early in that method.
> 
> The bug is rarely reproduced because in single screen configuration the main screen is usually 0, and in the multiscreen configuration Xinerama is usually active so the main screen is also 0 => the second part of the "if" statement is not executed.
> 
> I have validated the fix by the SwingSet, I also executed desktop tests in that config and found that even though this particular bug is fixed we still have many issues there, around ~100 tests failed.

This pull request has now been integrated.

Changeset: cfc9a881
Author:    Sergey Bylokhov <serb at openjdk.org>
URL:       https://git.openjdk.org/jdk19/commit/cfc9a881afd300bd7c1ce784287d1669308e89fc
Stats:     1 line in 1 file changed: 0 ins; 0 del; 1 mod

8288854: getLocalGraphicsEnvironment() on for multi-screen setups throws exception NPE

Reviewed-by: azvegint, aivanov

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

PR: https://git.openjdk.org/jdk19/pull/81



More information about the client-libs-dev mailing list