You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemml.apache.org by mb...@apache.org on 2017/07/08 22:19:19 UTC

systemml git commit: [SYSTEMML-1737] Fix resource leak in ParameterizedBuiltinCPFileInstruction

Repository: systemml
Updated Branches:
  refs/heads/master 0523cd490 -> f485ab292


[SYSTEMML-1737] Fix resource leak in ParameterizedBuiltinCPFileInstruction

Closes #564.


Project: http://git-wip-us.apache.org/repos/asf/systemml/repo
Commit: http://git-wip-us.apache.org/repos/asf/systemml/commit/f485ab29
Tree: http://git-wip-us.apache.org/repos/asf/systemml/tree/f485ab29
Diff: http://git-wip-us.apache.org/repos/asf/systemml/diff/f485ab29

Branch: refs/heads/master
Commit: f485ab292db3abf165ae530deb0bacee40a9143c
Parents: 0523cd4
Author: tedyu <yu...@gmail.com>
Authored: Sat Jul 8 15:15:08 2017 -0700
Committer: Matthias Boehm <mb...@gmail.com>
Committed: Sat Jul 8 15:15:08 2017 -0700

----------------------------------------------------------------------
 .../instructions/cpfile/ParameterizedBuiltinCPFileInstruction.java  | 1 +
 1 file changed, 1 insertion(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/systemml/blob/f485ab29/src/main/java/org/apache/sysml/runtime/instructions/cpfile/ParameterizedBuiltinCPFileInstruction.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/sysml/runtime/instructions/cpfile/ParameterizedBuiltinCPFileInstruction.java b/src/main/java/org/apache/sysml/runtime/instructions/cpfile/ParameterizedBuiltinCPFileInstruction.java
index 3046c7e..c8d890f 100644
--- a/src/main/java/org/apache/sysml/runtime/instructions/cpfile/ParameterizedBuiltinCPFileInstruction.java
+++ b/src/main/java/org/apache/sysml/runtime/instructions/cpfile/ParameterizedBuiltinCPFileInstruction.java
@@ -719,6 +719,7 @@ public class ParameterizedBuiltinCPFileInstruction extends ParameterizedBuiltinC
 				//Note: no need to handle empty result
 			}
 			finally {
+				IOUtilFunctions.closeSilently(fkeyMap);
 				IOUtilFunctions.closeSilently(twriter);
 				IOUtilFunctions.closeSilently(bwriter);
 			}