RE: RFR (S) JDK-8160174: java.net.NetworkInterface - fixes and improvements for network interface listing

2016-07-12 Thread Langer, Christoph
Chris, thanks for your comments. I'm kind of thinking the same - I actually also don't really like my result of merging the strings of code so much. I'll go on and prepare a smaller change set only addressing the real bugs. I also agree to switching the Linux implementation to getifaddrs at

Re: RFR 8151788: NullPointerException from ntlm.Client.type3

2016-07-12 Thread Weijun Wang
On 7/12/2016 22:34, Pavel Rappo wrote: What's the difference between no security buffer and an empty one (from the com.sun.security.ntlm.Client#type3's perspective)? I quickly browse through the NTLM protocol and yes they look like the same in each case. (Except for one which I am not sure,

Re: RFR 8151788: NullPointerException from ntlm.Client.type3

2016-07-12 Thread Pavel Rappo
What's the difference between no security buffer and an empty one (from the com.sun.security.ntlm.Client#type3's perspective)? > On 12 Jul 2016, at 15:25, Wang Weijun wrote: > > When there is no offset, there is no security buffer at all. When the length > is zero, the

Re: RFR 8151788: NullPointerException from ntlm.Client.type3

2016-07-12 Thread Wang Weijun
The fix looks fine to me. The readSecurityBuffer() is correct. When there is no offset, there is no security buffer at all. When the length is zero, the security buffer is an empty byte array. Here I think the customer encountered a server-side error. The fix is for interop with this

Re: RFR (S) JDK-8160174: java.net.NetworkInterface - fixes and improvements for network interface listing

2016-07-12 Thread Chris Hegarty
Christoph, > On 11 Jul 2016, at 14:36, Langer, Christoph wrote: > > Hi Chris (or anyone who is holding a stake in the NetworkInterface native > implementation), > > I’ve spent some time on cleaning up NetworkInterface.c and came up with a > bigger

Re: RFR 8151788: NullPointerException from ntlm.Client.type3

2016-07-12 Thread Vyom Tewari
gentile reminder. Vyom On Thursday 30 June 2016 02:16 PM, Vyom Tewari wrote: Hi All, Please review the below simple fix. Bug: JDK-8151788 NullPointerException from ntlm.Client.type3 Webrev : http://cr.openjdk.java.net/~vtewari/8151788/webrev0.0/index.html

Re: RFR 8144692: HttpServer API: use of non-existant method in example in package Javadoc

2016-07-12 Thread Vyom Tewari
Hi Pavel, Thanks for review, i updated the webrev(http://cr.openjdk.java.net/~vtewari/8144692/webrev0.0/index.html ) in place. Thanks, Vyom On Tuesday 12 July 2016 06:10 PM, Pavel Rappo wrote: Hi Vyom, A minor comment.

Re: RFR 8144692: HttpServer API: use of non-existant method in example in package Javadoc

2016-07-12 Thread Pavel Rappo
Hi Vyom, A minor comment. Do you think it would make sense to use space after new InetSocketAddress(8000), as in every other line in this javadoc where multiple arguments passed, the space is used. Just to be consistent in formatting. Thanks. > On 12 Jul 2016, at 12:36, Vyom Tewari

RFR 8144692: HttpServer API: use of non-existant method in example in package Javadoc

2016-07-12 Thread Vyom Tewari
Hi All, Please review below small doc fix. Bug: JDK-8144692 HttpServer API: use of non-existant method in example in package Javadoc Webrev : http://cr.openjdk.java.net/~vtewari/8144692/webrev0.0/index.html