Output stream close() leaks the file handle if flush or sync fails

Aberta Para iniciantes
  • Algo errado?
#234 0 comentários 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 3 dias

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
Menos de uma hora
Facilidade para iniciantes
88/100
Tipo de issue
Bug
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
java
Domínio
backend

Direção de pesquisa

Leia primeiro o método close() em core/src/main/java/com/backblaze/b2/util/B2FlushAndSyncFileOnCloseOutputStream.java (linhas 42-46), que chama em sequência out.flush(), out.getFD().sync() e out.close(). Depois, envolva as chamadas de flush e sync para que o stream subjacente seja fechado mesmo quando elas lançarem uma exceção. Está concluído quando o file descriptor é fechado em todos os caminhos e a IOException original continua sendo propagada.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

What happens

If close() is called on the stream and the flush or getFD().sync() call throws an IOException, the line that closes the underlying FileOutputStream is never reached. The caller gets the exception but the file descriptor stays open until garbage collection.

Where

core/src/main/java/com/backblaze/b2/util/B2FlushAndSyncFileOnCloseOutputStream.java:42-46, method close(): out.flush(); out.getFD().sync(); out.close(); with no try/finally.

Why it matters

Impact: Left behind (an open file handle), loud (the IOException is thrown). Severity: Medium, because it only happens in an unusual condition (an fsync or disk I/O error) and the handle is eventually released by finalization.

How to check

Found by reading the code, not reproduced. Read close(): if flush() or sync() throws, out.close() is skipped. A try/finally (or try-with-resources) around the first two calls would close the stream in every case.

Version

Seen at v6.5.0. Present on the default branch: present.

Linguagem predominante
Java
Estrelas
103
Forks
23
Merge médio
5d 8h
PRs com merge (30d)
3

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de Backblaze/b2-sdk-java

Todas as issues de Backblaze/b2-sdk-java

Issues semelhantes

Mais issues de Java

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.