文件描述符泄漏示例?

时间:2015-07-06 04:44:46

标签: java android memory-leaks file-descriptor memory-leak-detector

在Android中是否有任何好的示例来演示文件描述符泄漏?如果我们不关闭例如FileInputStreamFileOutputStream的流,我会在某处读到它,但我找不到任何可以证明它的好的参考示例。

请分享一些博客/代码段。谢谢!

3 个答案:

答案 0 :(得分:7)

因为Dalvik的 FileInputStream close itself when it is garbage collected(对于OpenJDK / Oracle也是如此),它实际上泄漏文件描述符的情况并不常见。当然,文件描述符将被“泄露”,直到GC运行,因此根据您的程序,它们可能需要一段时间才能被回收。

要实现更持久的泄漏,您必须通过在内存中的某处保留对它的引用来阻止流被垃圾收集。

这是一个简短的示例,每1秒加载一个属性文件,并在每次更改时跟踪它:

public class StreamLeak {

    /**
     * A revision of the properties.
     */
    public static class Revision {

        final ZonedDateTime time = ZonedDateTime.now();
        final PropertiesFile file;

        Revision(PropertiesFile file) {
            this.file = file;
        }
    }

    /*
     * Container for {@link Properties} that implements lazy loading.
     */
    public static class PropertiesFile {

        private final InputStream stream;
        private Properties properties;

        PropertiesFile(InputStream stream) {
            this.stream = stream;
        }

        Properties getProperties() {
            if(this.properties == null) {
                properties = new Properties();
                try {
                    properties.load(stream);
                } catch(IOException e) {
                    e.printStackTrace();
                }
            }
            return properties;
        }

        @Override
        public boolean equals(Object o) {
            if(o instanceof PropertiesFile) {
                return ((PropertiesFile)o).getProperties().equals(getProperties());
            }
            return false;
        }
    }

    public static void main(String[] args) throws IOException, InterruptedException {
        URL url = new URL(args[0]);
        LinkedList<Revision> revisions = new LinkedList<>();
        // Loop indefinitely
        while(true) {
            // Load the file
            PropertiesFile pf = new PropertiesFile(url.openStream());
            // See if the file has changed
            if(revisions.isEmpty() || !revisions.getLast().file.equals(pf)) {
                // Store the new revision
                revisions.add(new Revision(pf));
                System.out.println(url.toString() + " has changed, total revisions: " + revisions.size());
            }
            Thread.sleep(1000);
        }
    }
}

由于延迟加载,我们将 InputStream 保留在 PropertiesFile 中,每当我们创建新的 Revision 时都会保留 InputStream ,因为我们从不关闭流我们将在这里泄漏文件描述符。

现在,当程序终止时,操作系统将关闭这些打开的文件描述符,但只要程序运行,它就会继续泄漏文件描述符,如使用lsof所示:

$ lsof | grep pf.properties | head -n 3
java    6938   raniz   48r      REG    252,0    0    262694 /tmp/pf.properties
java    6938   raniz   49r      REG    252,0    0    262694 /tmp/pf.properties
java    6938   raniz   50r      REG    252,0    0    262694 /tmp/pf.properties
$ lsof | grep pf.properties | wc -l    
431

如果我们强制GC运行,我们可以看到大部分都返回了:

$ jcmd 6938 GC.run
6938:
Command executed successfully
$ lsof | grep pf.properties | wc -l
2

其余两个描述符是存储在 Revision 中的描述符。

我在我的Ubuntu机器上运行了这个,但如果在Android上运行,输出看起来会相似。

答案 1 :(得分:0)

InputStream in;
try {
    in = new BufferedInputStream(socket.getInputStream());

    // Do your stuff with the input stream
} catch (Exception e) {
    // Handle your exception
} finally {
    // Close the stream here
    if (in != null) {
        try {
            in.close();
        } catch (IOException e) {
            Log.e(TAG, "Unable to close stream: " + e);
        }
    }
}

想法是在finally块中关闭文件描述符。无论您是成功完成还是发生异常,都将正确关闭文件描述符。

现在,如果您正在寻找某些内容来演示如何不正确地执行此操作,请将此代码包装在while(1)循环中,注释掉in.close()行,然后放置{ {1}}在你的阻挡块中,这样当它爆炸时你就会突破你的无限循环。

答案 2 :(得分:-1)

InputStream in;

try {

    in = new FileInputStream(new File("abc");


    in.read(); // Do some stuff with open fileinputstream
    // If an exception is generated, inputstream object will not be closed
    // as the next statement will not be executed, instead jumping to 
    // the catch block. this will cause a leak of the fd assigned to file 
    // "abc" while opening it
    in.close()' 
  } catch (Exception e) {

    // Handle your exception

  }