RFR 8073542 : File Leak in jdk/src/java/base/unix/native/libnet/PlainDatagramSocketImpl.c

[Daniel Fuchs]

Hi Vyom,

I will sponsor your change and push the fix for you.
I'll sync up with Rob to check that we don't step on each other's toes.

best regards,

-- daniel

On 16/09/15 16:03, Chris Hegarty wrote:
> The changes look good to me Vyom.
>
> -Chris.
>
> On 16/09/15 10:08, Vyom Tewari wrote:
>> Hi All,
>>
>> Please review my changes for below bug.
>>
>> Bug:
>>        JDK-8073542 : File Leak in
>> jdk/src/java/base/unix/native/libnet/PlainDatagramSocketImpl.c
>> Webrev:
>> http://cr.openjdk.java.net/~dfuchs/vyom/8073542/webrev.00
>>
>> This change ensure that if "setsocketopt" fails we close the
>> corresponding fd and throw correct  exception.
>>
>> Thanks,
>> Vyom