<Beans Dev> [9] Review Request: 8146313 The java.beans.Statement.invoke() method handles 3-arg Class.forName incorrectly

Alexandr Scherbatiy alexandr.scherbatiy at oracle.com
Wed Jun 1 19:32:29 UTC 2016


The fix looks good to me.

Thanks,
Alexandr.

On 5/31/2016 6:41 PM, Sergey Bylokhov wrote:
> On 12.05.16 17:15, Sergey Bylokhov wrote:
>> Please review an updated version of the test, validation of the
>> exception when an invalid data was passed is added.
>>
>> http://cr.openjdk.java.net/~serb/8146313/webrev.04
>
> Does anybody have an additional comments/suggestions about this version?
>
>>
>>>
>>> --Semyon
>>>>
>>>>>>>>> On 4/22/2016 7:20 PM, Sergey Bylokhov wrote:
>>>>>>>>>> Hello.
>>>>>>>>>> Please review the fix for jdk9(the fix was discussed offline
>>>>>>>>>> already).
>>>>>>>>>>
>>>>>>>>>> The Statement class can execute the methods and pass some
>>>>>>>>>> parameters
>>>>>>>>>> for them. It can be used to store the statement(method
>>>>>>>>>> call+parameter)
>>>>>>>>>> in xml and load it when necessary. This class has special 
>>>>>>>>>> case for
>>>>>>>>>> Class.forName() because it provides some additional features(for
>>>>>>>>>> example it uses a different class loaders and supports primitive
>>>>>>>>>> types). But it does not take into account that there is the 
>>>>>>>>>> second
>>>>>>>>>> 3-args forName() method. It was fixed in the change.
>>>>>>>>>>
>>>>>>>>>> 3-args forName() method is considered as a usual method which
>>>>>>>>>> will be
>>>>>>>>>> executed for provided class+classloadder. The additional 
>>>>>>>>>> check for
>>>>>>>>>> package access is added. Note that I added a code to always
>>>>>>>>>> clone the
>>>>>>>>>> list of arguments just as an additional safeness for the future
>>>>>>>>>> changes.
>>>>>>>>>>
>>>>>>>>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8146313
>>>>>>>>>> Webrev can be found at:
>>>>>>>>>> http://cr.openjdk.java.net/~serb/8146313/webrev.03
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>>>>
>>>>
>>>
>>
>>
>
>



More information about the beans-dev mailing list