Integrated: 8300167: Add validation of the raster's layout before using in native

Sergey Bylokhov serb at openjdk.org
Thu Jan 19 05:05:37 UTC 2023


On Sat, 14 Jan 2023 07:23:18 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:

> Currently, we always validate image layouts passed from java to native, except for some types of rasters. One of the reasons why the rasters are skipped is that they are not public and have their own validation during construction. But it is unclear how it works in the multithreaded case. All these objects are not thread-safe - could it be possible that we can get the raster that is not completely constructed/validated?
> 
> FYI I have started to look into this method because when I did an accidental typo in pixel format in this method I got the same symptoms/crash as described [here](https://bugs.openjdk.org/browse/JDK-8278101). It might be unrelated but it is safer to fix.

This pull request has now been integrated.

Changeset: 7348b9ec
Author:    Sergey Bylokhov <serb at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/7348b9ec9373746bb76bc9fa9556f1811bd9e475
Stats:     9 lines in 1 file changed: 5 ins; 1 del; 3 mod

8300167: Add validation of the raster's layout before using in native

Reviewed-by: prr

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

PR: https://git.openjdk.org/jdk/pull/12004



More information about the client-libs-dev mailing list