On Fri, 19 Jan 2024 01:37:18 GMT, Julian Waters <jwat...@openjdk.org> wrote:
>> src/java.security.jgss/windows/native/libsspi_bridge/sspi.cpp line 422:
>>
>>> 420: gss_name_struct* name = new gss_name_struct;
>>> 421: if (name == nullptr) {
>>> 422: goto err;
>>
>> Go back? This looks more evil to me. Can we just `delete` and `return` here?
>
> I left it in there since it seemed inconsistent to have one site return while
> the rest used goto, I can replace it if needed
My understanding is that this is already after the `execution` label so it's a
different.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/16682#discussion_r1459391349
