[Bug] Static fileIoExecutor causes thread leak and StrictMode violations
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 52/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Domaine
- mobile, performance
Piste de recherche
Commencez dans com/github/moduth/blockcanary/BlockCanary.java en examinant l’initialisation statique de fileIoExecutor et le cycle de vie de stop(). Exercez le chemin de stop pendant les tests d’intégration et vérifiez que le thread de File-IO est terminé sans avertissements de fuite de thread de StrictMode.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Description
The BlockCanary class initializes a private static final Executor named fileIoExecutor for handling log zipping and writing. However, this executor is never shut down, even when BlockCanary.stop() is invoked.
Root Cause
-
Static Resource: The executor is defined as private static final at line 133.
-
Missing Lifecycle Management: There is no mechanism to terminate the fileIoExecutor. The stop() method (Line 103) only stops the CPU and Stack samplers but leaves the IO thread pool active.
-
Process-Level Scope: In Android, this static thread persists as long as the application process is alive, which is broader than the lifecycle of the monitoring session.
Impact
-
Thread Leak: The thread named "File-IO" persists indefinitely, leaking resources.
-
StrictMode Violations: This causes StrictMode thread leak warnings (e.g., Detecting explicit GC) during integration testing or when the app attempts to exit cleanly.
Relevant Code
File: com/github/moduth/blockcanary/BlockCanary.java
// Line 133
private static final Executor fileIoExecutor = newSingleThreadExecutor("File-IO");
// Line 103: stop() does not handle the executor
public void stop() {
if (mMonitorStarted) {
// ...
mBlockCanaryCore.stackSampler.stop();
mBlockCanaryCore.cpuSampler.stop();
// fileIoExecutor is left running
}
}
Suggested Fix
Modify the lifecycle of fileIoExecutor so it can be shut down inside the stop() method, or provide an explicit release() API to terminate the underlying thread pool.
- Langage dominant
- Java
- Étoiles
- 6.7k
- Forks
- 1k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
Nous n'avons pas encore vérifié les fichiers d'installation de ce projet. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de markzhai/AndroidPerformanceMonitor
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
markzhai/AndroidPerformanceMonitor#144 · 4 commentaires · 1 réaction ·
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 5/100
markzhai/AndroidPerformanceMonitor#153 · 5 commentaires · 1 réaction ·
-
这个库希望有人能维护一下Ouverte
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 15/100
markzhai/AndroidPerformanceMonitor#149 · 5 commentaires · 3 réactions ·
-
如何保证捕获卡顿堆栈的准确性？Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
-
android12崩溃，PendingIntent标志位需要适配Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
markzhai/AndroidPerformanceMonitor#147 · 8 commentaires ·
Toutes les issues de markzhai/AndroidPerformanceMonitor
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
Les mainteneurs répondent en général sous 1 jour
-
Content
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
RunestoneInteractive/rs#1559 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 86/100
inu-appcenter/memorIN-backend#298 ·
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
opendataloader-project/opendataloader-pdf#757 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
redhat-developer/intellij-quarkus#1626 ·