Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Backport JDK-8048782: OpenJDK: PiscesCache : xmax/ymax rounding up can cause RasterFormatException #94

Merged
merged 1 commit into from Mar 21, 2019

Conversation

Projects
None yet
2 participants
@sci-aws
Copy link
Contributor

commented Mar 20, 2019

In reference to issue: #93

Describe the bug
JDK-8048782 describes an issue where sun.java2d.pisces.PiscesCache constructor that accepts min/max x and y arguments - the internal 'bboxX1' and 'bboxY1' are set to values one greater than given maximum X and Y values. This effectively causes an "off by 1" error.

The fix has already been backported to OpenJDK9 and should also be brought back to OpenJDK8 as we've experienced this issue first hand.

References:
https://bugs.openjdk.java.net/browse/JDK-8048782
http://mail.openjdk.java.net/pipermail/2d-dev/2014-June/004652.html

To Reproduce
An sample application in the JBS issue is available, along with a JTREG test in the patch.

Expected behavior
The sample program and JTREG should pass.

Platform information
All supported platforms and OS versions.

Additional context
Although this has been back ported in OpenJDK8 and will be available for u222 we have experienced this issue first hand and need to be pro-active in making this fix available now.

@sci-aws sci-aws added the bug label Mar 20, 2019

@sci-aws sci-aws requested a review from bernd-aws Mar 20, 2019

@sci-aws sci-aws force-pushed the sci-aws:JDK-8048782 branch from f61bba1 to 228e075 Mar 21, 2019

@bernd-aws bernd-aws merged commit 57da39c into corretto:develop Mar 21, 2019

1 check passed

AWS CodeBuild us-west-2 (Corretto8_Github) Build succeeded for project Corretto8_Github
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.