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