On Fri, 25 Oct 2019 20:18:48 GMT, Phil Race <p...@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