[8u-dev] Request for Review & Approval to backport: 8073542: File Leak in jdk/src/java/base/unix/native/libnet/PlainDatagramSocketImpl.c

2016-05-29 Thread Ivan Gerasimov

As this has to be re-reviewed, sending the request to net-dev list as well.


On 27.05.2016 16:46, Ivan Gerasimov wrote:

Hello!


I'd like to backport the fix to jdk8u-dev.

The fix had to be slightly adjusted due to the different way the error 
messages are retrieved in jdk8.


Here's the webrev with modified fix:
http://cr.openjdk.java.net/~igerasim/8073542/00/webrev/


Bug: https://bugs.openjdk.java.net/browse/JDK-8073542
Jdk9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/bb6e5a409fef
Jdk9 review: 
http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035223.html





With kind regards,
Ivan



Re: [8u-dev] Request for Review & Approval to backport: 8073542: File Leak in jdk/src/java/base/unix/native/libnet/PlainDatagramSocketImpl.c

2016-06-06 Thread Ivan Gerasimov

Could someone please help review this simple backport?


Thanks in advance,

Ivan


On 29.05.2016 21:08, Ivan Gerasimov wrote:
As this has to be re-reviewed, sending the request to net-dev list as 
well.



On 27.05.2016 16:46, Ivan Gerasimov wrote:

Hello!


I'd like to backport the fix to jdk8u-dev.

The fix had to be slightly adjusted due to the different way the 
error messages are retrieved in jdk8.


Here's the webrev with modified fix:
http://cr.openjdk.java.net/~igerasim/8073542/00/webrev/


Bug: https://bugs.openjdk.java.net/browse/JDK-8073542
Jdk9 changeset: http://hg.openjdk.java.net/jdk9/dev/jdk/rev/bb6e5a409fef
Jdk9 review: 
http://mail.openjdk.java.net/pipermail/core-libs-dev/2015-September/035223.html





With kind regards,
Ivan