[PING] RFR: JDK-8135259: InetAddress.getAllByName only reports "unknown error" instead of actual cause

[Ramanand Patil]

Hi Sean,

Thank you. I have updated copyright year to the earlier patch and added a simple test case to cover this bug.
Please review the updated Webrev: http://cr.openjdk.java.net/~rpatil/8135259/webrev.01/

Regards,
Ramanand.

-----Original Message-----
From: Seán Coffey 
Sent: Wednesday, February 10, 2016 7:47 PM
To: Ivan Gerasimov; Ramanand Patil; OpenJDK Network Dev list
Subject: Re: [PING] RFR: JDK-8135259: InetAddress.getAllByName only reports "unknown error" instead of actual cause

(dropping core-libs)

src change looks good to me also Ramanand. Will you also be adding a testcase to cover this ? I'm sure you could find one in the test base for editing.

Regards,
Sean.

On 10/02/16 13:44, Ivan Gerasimov wrote:
> Hi Ramanand!
>
> Your fix looks good to me.
>
> I'm forwarding your request to net-dev at openjdk.java.net which is 
> probably more appropriate to review this fix.
> It would be good, if net-dev people can confirm they're Okay with the 
> fix.
>
> Sincerely yours,
> Ivan
>
>
> On 10.02.2016 10:08, Ramanand Patil wrote:
>> Hi all,
>>
>> Please help me in getting this review done.
>>
>> Regards,
>> Ramanand.
>>
>> -----Original Message-----
>> From: Ramanand Patil
>> Sent: Friday, February 05, 2016 10:46 PM
>> To: core-libs-dev at openjdk.java.net
>> Subject: RFR: JDK-8135259: InetAddress.getAllByName only reports 
>> "unknown error" instead of actual cause
>>
>> Hi all
>>   Please review the fix for bug: 
>> https://bugs.openjdk.java.net/browse/JDK-8135259
>>
>> Bug Description: Attempts to resolve a host unknown to the DNS server 
>> cause an UnknownHostException stating "unknown error" instead of 
>> "Name or service not known".
>>
>> Webrev: http://cr.openjdk.java.net/~rpatil/8135259/webrev.00/
>>
>> Fix: Using a function call directly "gai_strerror(gai_error)" instead 
>> of using the function pointer which was declared but not initialized.
>> Apart from this I have removed few other unused variables and 
>> function pointers. Brief description of how this bug was introduced 
>> is added to the bug comment.
>>
>> Regards,
>>
>> Ramanand.
>>
>>
>