Re: [commons-bcel] branch master updated: Unknown attributes with invalid length now trigger ClassFormatException

2022-11-21 Thread Gary Gregory
Hi Mark, Any reason not to do this check in the Attribute superclass? Gaty On Mon, Nov 21, 2022, 05:50 wrote: > This is an automated email from the ASF dual-hosted git repository. > > markt pushed a commit to branch master > in repository https://gitbox.apache.org/repos/asf/commons-bcel.git >

Re: [commons-bcel] branch master updated: Unknown attributes with invalid length now trigger ClassFormatException

2022-11-21 Thread Gary D. Gregory
Hm, after reading https://docs.oracle.com/javase/specs/jvms/se7/html/jvms-4.html#jvms-4.7 I will pull up the validation. TY for the commit :-) Gary On 2022/11/21 12:00:20 Gary Gregory wrote: > Hi Mark, > > Any reason not to do this check in the Attribute superclass? > > Gaty > > On Mon, Nov

Re: [commons-bcel] branch master updated: Validate the u4 length of all attributes

2022-11-21 Thread Gary Gregory
Gotcha, will fix. Gary On Mon, Nov 21, 2022, 08:23 Mark Thomas wrote: > On 21/11/2022 13:03, ggreg...@apache.org wrote: > > This is an automated email from the ASF dual-hosted git repository. > > > > ggregory pushed a commit to branch master > > in repository https://gitbox.apache.org/repos/asf