-
-
Notifications
You must be signed in to change notification settings - Fork 149
Closed
Labels
has-failing-testIndicates that there exists a test case (under `failing/`) to reproduce the issueIndicates that there exists a test case (under `failing/`) to reproduce the issueion
Milestone
Description
(note: found via https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=65180)
Looks like overflow checks not being by IonParser can result in one of outcomes:
- an Ion
RuntimeExceptionbeing thrown, when methodgetIntValue()is called with value beyond 32-bitintrange - quiet truncation of value without failure
Instead, this should be surfaced same as equivalent JsonParseException JSON-backed JsonParser throws in similar situation -- see ParserBase.convertNumberToInt() (and ParserBase.reportOverflowInt()) for examples of handling.
Same probably affects long overflow as well (wrt BigInteger range value)
Metadata
Metadata
Assignees
Labels
has-failing-testIndicates that there exists a test case (under `failing/`) to reproduce the issueIndicates that there exists a test case (under `failing/`) to reproduce the issueion