FTPUtil 存在内存泄漏的bug?

Question

private boolean uploadFile(String remotePath,List<File> fileList) throws IOException {
        boolean uploaded = true;
        FileInputStream fis = null;
        //连接FTP服务器
        if(connectServer(this.ip,this.port,this.user,this.pwd)){
            try {
                ftpClient.changeWorkingDirectory(remotePath);
                ftpClient.setBufferSize(1024);
                ftpClient.setControlEncoding("UTF-8");
                ftpClient.setFileType(FTPClient.BINARY_FILE_TYPE);
                ftpClient.enterLocalPassiveMode();
                for(File fileItem : fileList){
                    fis = new FileInputStream(fileItem);
                    ftpClient.storeFile(fileItem.getName(),fis);
                }

            } catch (IOException e) {
                logger.error("上传文件异常",e);
                uploaded = false;
                e.printStackTrace();
            } finally {
                fis.close();
                ftpClient.disconnect();
            }
        }
        return uploaded;
    }



    private boolean connectServer(String ip,int port,String user,String pwd){

        boolean isSuccess = false;
        ftpClient = new FTPClient();
        try {
            ftpClient.connect(ip);
            isSuccess = ftpClient.login(user,pwd);
        } catch (IOException e) {
            logger.error("连接FTP服务器异常",e);
        }
        return isSuccess;
    }

如果ftpClient.connect(ip);这句抛出异常,connectServer方法返回的结果是false, 然后ftpClient.disconnect();这句就不会被执行。

而且fis.close();这句没有对fis进行非空判断，fis也有很大概率是null的情况，如果fis.close();这句抛出空指针异常（或者其他异常），那么ftpClient.disconnect();这句也不会被执行了。

不知道我分析的是否有道理~

Answer 1

Geely 2018-06-03 20:10:58

你好，同学，这个不会内存泄露，听你的意思是，关闭连接资源的问题。如果更好一点的是，可以把connectserver异常向上抛出即可。因为在upload里面有finally来关闭。

正常来说连接已经失败，这个连接也就会自然被JVM回收。因为不会再使用他，重新上传的时候也会重新去建立一个新的连接~~

• 提问者 qq_谁动了我的奶酪_03546962 #1

  好的 原来是这样

Answer 2

慕粉1433416698 2018-10-08 06:30:59

如果是上传多个File的话，finally中只关闭了最后一个fis.close()； 所以还是会内存泄露

Answer 3

慕神8589343 2018-05-29 23:36:13

ftpClient.connect(ip)都异常了，你还disconnect神马

• 提问者 qq_谁动了我的奶酪_03546962 #1

  好吧 我主要想表达的意思是ftpClient.disconnect();在很多场景会不被执行。假设ftpClient.connect(ip)不抛异常，密码错误登录失败  isSuccess = ftpClient.login(user,pwd); 这个地方isSuccess 是不是false？ftpClient.disconnect();是不会被执行吧？