-
Bug
-
Resolution: Fixed
-
P3
-
25
-
None
-
master
-
generic
-
generic
The patch for JDK-8357299 does not completely fix the regression it was intended to address. Instead of bailing out on overflow, it introduces additional logic around src and clip, which might lead to attempts to draw invalid source pixels.
A test case demonstrating the issue is attached to JDK-8358103(passed on jdk22 failed on latest openjdk/jdk).
I believe it would be better to bail out early in case of overflow than risk incorrect calculations. There are also some unresolved discussions related to that patch.
Therefore, I propose revertingJDK-8357299 until a proper fix is available. This approach would also simplify backports, since only the patch for JDK-8358103 would need to be backported.
A test case demonstrating the issue is attached to JDK-8358103(passed on jdk22 failed on latest openjdk/jdk).
I believe it would be better to bail out early in case of overflow than risk incorrect calculations. There are also some unresolved discussions related to that patch.
Therefore, I propose reverting
- relates to
-
JDK-8357299 Graphics copyArea doesn't copy any pixels when there is overflow
-
- Resolved
-
- links to
-
Commit(master) openjdk/jdk/d9d00d33
-
Review(master) openjdk/jdk/25537