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

[Ivan Gerasimov]

Hi Vyom!

The change looks fine, thanks.

You've used strerr() to extract the error string, so it may be good to 
coordinate your patch with the fix for JDK-8133249.
Robert (CCed) is working on it at the moment.

Sincerely yours,
Ivan

On 16.09.2015 12: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
>