RFR: 8361521 : BogusFocusableWindowState.java fails with StackOverflowError on Linux [v5]

Sergey Bylokhov serb at openjdk.org
Tue Aug 26 00:54:50 UTC 2025


On Wed, 20 Aug 2025 05:05:20 GMT, Anass Baya <abaya at openjdk.org> wrote:

>> **Analysis:**
>> The getFocusableWindowState() method is not intended to modify the configuration; doing so can cause recursive re-entry on Linux.
>> 
>> **Proposed Fix:**
>> We are intentionally overriding getFocusableWindowState() to allow it to change the configuration, in order to verify that calling getScreenImOn() for ownerless windows does not throw any exceptions.
>> To prevent recursive re-entry, we use a flag to ensure that getFocusableWindowState() is executed only once
>
> Anass Baya has updated the pull request incrementally with one additional commit since the last revision:
> 
>   add space

looks fine, please confirm that all tests are green.

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

Marked as reviewed by serb (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/26298#pullrequestreview-3153452042


More information about the client-libs-dev mailing list