On 31/10/2018 19:26, Brian Burkhalter wrote:

On Oct 31, 2018, at 12:20 PM, Brian Burkhalter <brian.burkhal...@oracle.com <mailto:brian.burkhal...@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

Reply via email to