RFR: 8305089: Implement missing socket options on AIX [v4]

Varada M duke at openjdk.org
Fri Apr 14 12:32:38 UTC 2023


On Fri, 14 Apr 2023 12:13:39 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:

>> Varada M has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains six commits:
>> 
>>  - Merge branch 'master' into aix_socket_options
>>  - Updated copyright year
>>  - Update to copyright year
>>  - AIX socket options
>>  - aix socket options
>>  - AIX Socket Options
>
> src/jdk.net/aix/native/libextnet/AIXSocketOptions.c line 133:
> 
>> 131:     } else {
>> 132:         if ((int)cred_info.euid == -1) {
>> 133:             handleError(env, -1, "get  failed");
> 
> Suggestion:
> 
>             handleError(env, -1, "get SO_PEERID failed");

Thanks @dfuch and @jaikiran. I have updated to `"get SO_PEERID failed"`

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

PR Review Comment: https://git.openjdk.org/jdk/pull/13240#discussion_r1166772753



More information about the build-dev mailing list