8213210: Change ServerSocket(SocketImpl impl) constructor to protected access
Alan Bateman
Alan.Bateman at oracle.com
Wed Oct 31 19:32:45 UTC 2018
On 31/10/2018 19:26, Brian Burkhalter wrote:
>
>> On Oct 31, 2018, at 12:20 PM, Brian Burkhalter
>> <brian.burkhalter at oracle.com <mailto:brian.burkhalter at oracle.com>> wrote:
>>
>> - ServerSocket(SocketImpl impl) {
>> + protected ServerSocket(SocketImpl impl) {
>> this.impl = impl;
>> impl.setServerSocket(this); // <- NPE if impl == null
>> }
>
> Oops, it can throw null as indicated above. Revised patch below.
>
Good, I think looks okay.
-Alan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/net-dev/attachments/20181031/0392aebd/attachment.html>
More information about the net-dev
mailing list