[ARROW-10017] [Java] LargeMemoryUtil.checkedCastToInt has buggy logic

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0.1
• Fix Version/s: 2.0.0
• Component/s: Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/26041
• Assignee: Unassigned
• Reporter: Patrick Woody
• Created: 15/Sep/20 14:03
• Updated: 11/Jan/23 08:10
• Resolved: 21/Sep/20 03:28

Description

LargeMemoryUtil.checkedCastToInt does checked conversion from long -> int if BoundsChecking is enabled. However, it looks like there is a bug in the logic:

length <= Integer.MAX_VALUE || length >= Integer.MIN_VALUE

is always true.

Issue Links

• links to: GitHub Pull Request #8194