RFR: 8272096: Exceptions::new_exception can return wrong exception [v3]

Coleen Phillimore coleenp at openjdk.org
Mon Jul 18 15:10:47 UTC 2022


On Fri, 15 Jul 2022 15:09:07 GMT, Coleen Phillimore <coleenp at openjdk.org> wrote:

>> I added an assert if Exceptions::new_exception is called with a pending exception and fixed the places where it is called with a pending exception.   That leaves only two possible exceptions. I left the product mode code in to return the pending exception if allocating the exception message doesn't thrown OOM because it was always there and seems dubious.  Tested with jck tests and tier1-7.
>
> Coleen Phillimore has updated the pull request incrementally with one additional commit since the last revision:
> 
>   second get_user_name_slow call should CHECK_NULL too.

I actually haven't changed the behavior because instead of "bad class index" in the case of truncation, and the old behavior would throw the truncation exception.
It is more correct.

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

PR: https://git.openjdk.org/jdk/pull/9492


More information about the hotspot-dev mailing list