[Approved] RFR: 8189092: ArrayIndexOutOfBoundsException on Linux in getCachedGlyph

[Kevin Rushforth]

On Fri, 25 Oct 2019 20:18:48 GMT, Phil Race <prr at openjdk.org> wrote:

> I have added an evaluation in https://bugs.openjdk.java.net/browse/JDK-8207839
> Please read that for more detail, but basically we are not properly preventing or
> handling cases where fontconfig causes us to overflow the byte storage used
> for a font "slot".
> 
> ----------------
> 
> Commits:
>  - b6c19466: Fix whitespace
>  - e00ef992: 8189092: ArrayIndexOutOfBoundsException on Linux in getCachedGlyph
> 
> Changes: https://git.openjdk.java.net/jfx/pull/24/files
>  Webrev: https://webrevs.openjdk.java.net/jfx/24/webrev.00
>   Issue: https://bugs.openjdk.java.net/browse/JDK-8189092
>   Stats: 15 lines in 3 files changed: 12 ins; 0 del; 3 mod
>   Patch: https://git.openjdk.java.net/jfx/pull/24.diff
>   Fetch: git fetch https://git.openjdk.java.net/jfx pull/24/head:pull/24

Looks good to me. I don't think this needs a second review, although @bourgesl or @johanvos might want to weigh in.

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

Approved by kcr (Lead).

PR: https://git.openjdk.java.net/jfx/pull/24