RFR: 8264142: Remove TRAPS/THREAD parameters for verifier related functions [v4]

Harold Seigel hseigel at openjdk.java.net
Fri Mar 26 17:19:46 UTC 2021


On Fri, 26 Mar 2021 05:04:55 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Harold Seigel has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Address dholmes's comments
>
> src/hotspot/share/classfile/verifier.cpp line 695:
> 
>> 693:   ConstantPool* cp = _klass->constants();
>> 694:   Symbol* const method_sig = cp->symbol_at(sig_index);
>> 695:   translate_signature(method_sig, sig_verif_types, CHECK_VERIFY(this));
> 
> CHECK_VERIFY is not directly related to exceptions. Are there really no verification errors possible from all of these calls where CHECK_VERIFY has been removed?

The calls where CHECK_VERIFY's were removed are to functions that deal with signatures.  These methods do not check for errors because the signatures have already been verified by classFileParser.cpp in functions such as verify_legal_method_signature() and verify_legal_field_signature().

-------------

PR: https://git.openjdk.java.net/jdk/pull/3194


More information about the hotspot-runtime-dev mailing list