6791458: FileInputStream/RandomAccessFile.read leaks memory if invoked on closed stream with len > 8k
Reviewed-by: alanb
Contributed-by: jeremymanson@google.com
--- a/jdk/src/share/native/java/io/io_util.c Wed Jan 07 11:50:32 2009 -0800
+++ b/jdk/src/share/native/java/io/io_util.c Thu Jan 08 14:07:45 2009 -0800
@@ -105,18 +105,18 @@
fd = GET_FD(this, fid);
if (fd == -1) {
JNU_ThrowIOException(env, "Stream Closed");
- return -1;
- }
-
- nread = IO_Read(fd, buf, len);
- if (nread > 0) {
- (*env)->SetByteArrayRegion(env, bytes, off, nread, (jbyte *)buf);
- } else if (nread == JVM_IO_ERR) {
- JNU_ThrowIOExceptionWithLastError(env, "Read error");
- } else if (nread == JVM_IO_INTR) { /* EOF */
- JNU_ThrowByName(env, "java/io/InterruptedIOException", 0);
- } else { /* EOF */
nread = -1;
+ } else {
+ nread = IO_Read(fd, buf, len);
+ if (nread > 0) {
+ (*env)->SetByteArrayRegion(env, bytes, off, nread, (jbyte *)buf);
+ } else if (nread == JVM_IO_ERR) {
+ JNU_ThrowIOExceptionWithLastError(env, "Read error");
+ } else if (nread == JVM_IO_INTR) { /* EOF */
+ JNU_ThrowByName(env, "java/io/InterruptedIOException", 0);
+ } else { /* EOF */
+ nread = -1;
+ }
}
if (buf != stackBuf) {
--- /dev/null Thu Jan 01 00:00:00 1970 +0000
+++ b/jdk/test/java/io/readBytes/MemoryLeak.java Thu Jan 08 14:07:45 2009 -0800
@@ -0,0 +1,51 @@
+/*
+ * Copyright 2008 Sun Microsystems, Inc. All Rights Reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Sun Microsystems, Inc., 4150 Network Circle, Santa Clara,
+ * CA 95054 USA or visit www.sun.com if you need additional information or
+ * have any questions.
+ */
+
+/*
+ * @test
+ * @bug 6791458
+ * @summary Reading from closed input files leaks native memory
+ */
+
+import java.io.*;
+
+public class MemoryLeak {
+ public static void main(String[] args) throws Throwable {
+ byte[] bytes = new byte[1 << 20];
+ String dir = System.getProperty("test.src", ".");
+ File testFile = new File(dir, "input.txt");
+ FileInputStream s = new FileInputStream(testFile);
+ s.close();
+ for (int i = 0; i < 10000; i++) {
+ try {
+ s.read(bytes);
+ throw new Error("expected IOException");
+ } catch (IOException _) {
+ /* OK */
+ } catch (OutOfMemoryError oome) {
+ System.out.printf("Got OutOfMemoryError, i=%d%n", i);
+ throw oome;
+ }
+ }
+ }
+}