Close ObjectOutputStream in a finally clause

Fixes junit-team#1556.
aristotle0x01 · Oct 3, 2018 · 4c4663f · 4c4663f
1 parent 144b084
commit 4c4663f
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/src/main/java/org/junit/experimental/max/MaxHistory.java b/src/main/java/org/junit/experimental/max/MaxHistory.java
@@ -75,10 +75,15 @@ private MaxHistory(File storedResults) {
     }
 
     private void save() throws IOException {
-        ObjectOutputStream stream = new ObjectOutputStream(new FileOutputStream(
-                fHistoryStore));
-        stream.writeObject(this);
-        stream.close();
+        ObjectOutputStream stream = null;
+        try {
+            stream = new ObjectOutputStream(new FileOutputStream(fHistoryStore));
+            stream.writeObject(this);
+        } finally {
+            if (stream != null) {
+                stream.close();
+            }
+        }
     }
 
     Long getFailureTimestamp(Description key) {