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

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#234 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 3 jours

Personne n'a encore pris cette issue.

Évaluation

Difficulté
1/5
Temps estimé
Moins d'une heure
Accessibilité débutants
88/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
java
Domaine
backend

Piste de recherche

Commence par lire la méthode close() de core/src/main/java/com/backblaze/b2/util/B2FlushAndSyncFileOnCloseOutputStream.java (lignes 42-46), qui appelle successivement out.flush(), out.getFD().sync() et out.close(). Encadre ensuite les appels à flush et sync pour que le flux sous-jacent soit fermé même lorsqu'ils lèvent une exception. Le travail est terminé lorsque le file descriptor est fermé sur tous les chemins d'exécution et que l'IOException d'origine continue d'être propagée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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.

Langage dominant
Java
Étoiles
103
Forks
23
Merge moyen
5 j 8 h
PR mergées (30 j)
3

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de Backblaze/b2-sdk-java

Toutes les issues de Backblaze/b2-sdk-java

Issues similaires

Plus d'issues Java

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.