[ https://issues.apache.org/jira/browse/HADOOP-10517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13989274#comment-13989274 ]
Swarnim Kulkarni commented on HADOOP-10517: ------------------------------------------- That's very interesting because the only thing extra that closeQuietly does is check for null which anyways should throw an NPE for a null value > InputStream is not closed in two methods of JarFinder > ----------------------------------------------------- > > Key: HADOOP-10517 > URL: https://issues.apache.org/jira/browse/HADOOP-10517 > Project: Hadoop Common > Issue Type: Bug > Reporter: Ted Yu > Assignee: Ted Yu > Priority: Minor > Attachments: HADOOP-10517.1.patch.txt, hadoop-10517-v1.txt, > hadoop-10517-v2.txt > > > JarFinder#jarDir() and JarFinder#zipDir() have such code: > {code} > InputStream is = new FileInputStream(f); > copyToZipStream(is, anEntry, zos); > {code} > The InputStream is closed in copyToZipStream() but should be enclosed in > finally block. -- This message was sent by Atlassian JIRA (v6.2#6252)