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