Hello,

I am not sure about the JNi conventions, but does the jObjectArray rv need to 
be released as local reference in some of the early returns as well?

Gruss
Bernd

Gruss
Bernd
--
http://bernd.eckenfels.net

________________________________
Von: -1031249216m Auftrag von
Gesendet: Mittwoch, Juli 18, 2018 4:38 AM
An: Ivan Gerasimov
Cc: core-libs-dev
Betreff: Re: RFR 8207750 : Native handle leak in java.io.WinNTFileSystem.list()

Hi Ivan,

This looks all right to me.

Thanks,

Brian

On Jul 17, 2018, at 5:01 PM, Ivan Gerasimov <ivan.gerasi...@oracle.com> wrote:

> In the file WinNTFileSystem_md.c there is a potential leak of native handle 
> obtained from FindFirstFileW() in the unfortunate event of memory allocation 
> failure.
>
> Would you please help review a simple fix?
>
> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8207750
> WEBREV:http://cr.openjdk.java.net/~igerasim/8207750/00/webrev/

Reply via email to