RFR: 8201567: QuantumRenderer modifies buffer in use by JavaFX Application Thread

Kevin Rushforth kcr at openjdk.java.net
Sat Jun 27 15:40:05 UTC 2020


On Fri, 26 Jun 2020 03:47:55 GMT, John Neffenger <github.com+1413266+jgneff at openjdk.org> wrote:

> Fixes [JDK-8201567](https://bugs.openjdk.java.net/browse/JDK-8201567).

At first glance the fix looks fine (aside from the `assert` statements that need to be removed). It will also need to
be tested using the SW pipeline on all platforms.

modules/javafx.graphics/src/main/java/com/sun/glass/ui/Pixels.java line 194:

> 193:     public final Buffer getBuffer() {
> 194:         assert this.bytes != null || this.ints != null;
> 195:         return this.bytes != null ? this.bytes : this.ints;

We should not use `assert` statements in the JavaFX runtime. If this is a condition that needs to be checked, then use
an `if` test and throw an appropriate `RuntimeException` or an `InternalError`.

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

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


More information about the openjfx-dev mailing list