On Tue, 21 Mar 2023 20:21:23 GMT, Chen Liang <li...@openjdk.org> wrote:
>> This typo doesn't allow creation of malformed ClassDesc or MethodTypeDesc, >> but it produces an erroneous exception on certain inputs. Running >> `java.lang.constant.MethodTypeDesc.ofDescriptor("(IIIII[[[V)I")` in Jshell >> 19.0.2 throws StringIndexOutOfBoundsException, and throws >> IllegalArgumentException in the Jshell for this patch. > > Chen Liang has updated the pull request with a new target base due to a merge > or a rebase. The incremental webrev excludes the unrelated changes brought in > by the merge/rebase. The pull request contains five additional commits since > the last revision: > > - Merge branch 'master' of https://git.openjdk.java.net/jdk into 8303930 > - Add test for skipOverFieldSignature bug > - Merge branch 'master' of https://git.openjdk.java.net/jdk into 8303930 > - Copyright year > - 8303930: Fix ConstantUtils.skipOverFieldSignature void case return value Could any engineer please review this patch fixing `ConstantUtils.skipOverFieldSignature` throwing `StringIndexOutOfBoundsException` on certain crafted inputs? ------------- PR Comment: https://git.openjdk.org/jdk/pull/12964#issuecomment-1480500845