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

Coleen Phillimore coleenp at openjdk.org
Tue Jul 19 14:44:52 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.

Thanks David for the review and for battling the confusion this code provoked.  Hopefully in the end this helps.

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

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


More information about the hotspot-dev mailing list