XS RFR: 8009731: loader constraint violation error message
Coleen Phillimore
coleen.phillimore at oracle.com
Mon Mar 25 17:47:15 PDT 2013
On 3/25/2013 8:10 PM, David Holmes wrote:
> Hi Karen,
>
> Looks good to me.
>
> On 26/03/2013 7:08 AM, Karen Kinnear wrote:
>>
>> Webrev: http://cr.openjdk.java.net/~acorn/8009731/webrev/
>
> I can't quite determine why the extra ResourceMark caused the observed
> symptoms. I would have expected something more dramatic if we returned
> a string that was already released - are we simply overwriting it with
> a later class name (hence the wrong name in the message)? Do we also
> know when this regression was introduced?
So that's why I said "Looks ok" rather than "Looks good." I think that
function should return a Symbol* because it's not obvious to the casual
observer that the as_C_string() string is resource allocated and the
ResourceMark destructor will reclaim the memory. I think the error
message might have been either garbled or pointing to a different
string. It could have been like that for a while and nobody noticed.
Coleen
>
>> Bug: https://jbs.oracle.com/bugs/browse/JDK-8009731
>
> This is not visible external to Oracle.
>
> Thanks,
> David
>
>> Fix loader constraint violation error message.
>>
>> tests:
>> bug report test
>> vm.quick.testlist in parallel
>>
More information about the hotspot-runtime-dev
mailing list