<i18n dev> RFR: 8350646: Calendar.Builder.build() Throws ArrayIndexOutOfBoundsException [v3]
Naoto Sato
naoto at openjdk.org
Wed Feb 26 20:03:02 UTC 2025
On Wed, 26 Feb 2025 19:48:10 GMT, Justin Lu <jlu at openjdk.org> wrote:
>> Please review this PR which prevents an `AIOOBE` from leaking out when `java.util.Calendar.Builder` is used to build a Japanese calendar with an era value too large.
>>
>> Note that we don't check under `BEFORE_MEIJI`/0 as historically Japanese calendar ignores negative values during normalization. See `JapaneseImperialCalendar` L2018: `date.setEra(era > 0 ? eras[era] : null);`.
>>
>> We also check against `eras.length` over `REIWA`/5 due to the possibility of additional eras via the property override or future eras in general.
>
> Justin Lu has updated the pull request incrementally with one additional commit since the last revision:
>
> Replace 7 w/ more meaningful value
Thanks. I reviewed the CSR as well.
-------------
Marked as reviewed by naoto (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/23789#pullrequestreview-2645746475
More information about the i18n-dev
mailing list