diff options
author | Abhishek Koneru <akoneru@redhat.com> | 2012-06-20 13:56:57 -0400 |
---|---|---|
committer | Endi Sukma Dewata <edewata@redhat.com> | 2012-06-25 13:38:26 -0500 |
commit | 8a8436f73d4c4edd10b43313b724f32ccb7b957e (patch) | |
tree | 657c8f9accd34d286acf0569f768c72684a12b75 /base/common/src/com/netscape/cmscore/util | |
parent | de3aaef15e9b1f192344019f52d6c80860055b5e (diff) | |
download | pki-8a8436f73d4c4edd10b43313b724f32ccb7b957e.tar.gz pki-8a8436f73d4c4edd10b43313b724f32ccb7b957e.tar.xz pki-8a8436f73d4c4edd10b43313b724f32ccb7b957e.zip |
Fixes for Coverity issues of type Resource Leaks - Remaining
Diffstat (limited to 'base/common/src/com/netscape/cmscore/util')
-rw-r--r-- | base/common/src/com/netscape/cmscore/util/FileAsString.java | 23 |
1 files changed, 13 insertions, 10 deletions
diff --git a/base/common/src/com/netscape/cmscore/util/FileAsString.java b/base/common/src/com/netscape/cmscore/util/FileAsString.java index 7853346f5..ea8fb796d 100644 --- a/base/common/src/com/netscape/cmscore/util/FileAsString.java +++ b/base/common/src/com/netscape/cmscore/util/FileAsString.java @@ -52,16 +52,19 @@ public class FileAsString { BufferedReader br = createBufferedReader(mFilename); StringBuffer buf = new StringBuffer(); int bytesread = 0; - - do { - char cbuf[] = new char[16]; - - bytesread = br.read(cbuf, 0, cbuf.length); - if (bytesread > 0) { - buf.append(cbuf, 0, bytesread); - } - } while (bytesread != -1); - br.close(); + try { + do { + char cbuf[] = new char[16]; + + bytesread = br.read(cbuf, 0, cbuf.length); + if (bytesread > 0) { + buf.append(cbuf, 0, bytesread); + } + } while (bytesread != -1); + } finally { + if (br != null) + br.close(); + } fileContents = new String(buf); } |