[lworld] RFR: 8340409: [lworld] Simple serialization and deserialization of core migrated classes [v2]

Viktor Klang vklang at openjdk.org
Wed Oct 9 21:22:24 UTC 2024


On Wed, 9 Oct 2024 18:14:03 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> src/java.base/share/classes/java/io/ObjectStreamClass.java line 1591:
>> 
>>> 1589:                         return null;
>>> 1590:                     }})
>>> 1591:                 .filter(m -> matchFactoryParamTypes(clazz, m, fields))
>> 
>> `m` can be null here, so `matchFactoryParamTypes` needs to check that.
>
> After setAccessible IAE should never happen; change to throw InternalError.

Yeah, sounds good. 👍

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

PR Review Comment: https://git.openjdk.org/valhalla/pull/1248#discussion_r1794243082


More information about the valhalla-dev mailing list