summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Narayan Kamath <narayan@google.com> 2015-06-25 08:33:49 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2015-06-25 08:33:50 +0000
commitfdb70dc4346f801673aa700f3fca2b8cc77b809c (patch)
tree62592a827185a72cc4a59d21f62359870889db2c
parenta78c0e0b9aff70d03101a25b47024f6d02cb93e2 (diff)
parente1856e482933b24e66f0f432ac0cb0157787e9ce (diff)
Merge "Fix filedescriptor leakage in MediaMetadataRetriever"
-rw-r--r--media/java/android/media/MediaMetadataRetriever.java10
1 files changed, 1 insertions, 9 deletions
diff --git a/media/java/android/media/MediaMetadataRetriever.java b/media/java/android/media/MediaMetadataRetriever.java
index 9a69c067e498..9a73dfcd275b 100644
--- a/media/java/android/media/MediaMetadataRetriever.java
+++ b/media/java/android/media/MediaMetadataRetriever.java
@@ -64,9 +64,7 @@ public class MediaMetadataRetriever
throw new IllegalArgumentException();
}
- FileInputStream is = null;
- try {
- is = new FileInputStream(path);
+ try (FileInputStream is = new FileInputStream(path)) {
FileDescriptor fd = is.getFD();
setDataSource(fd, 0, 0x7ffffffffffffffL);
} catch (FileNotFoundException fileEx) {
@@ -74,12 +72,6 @@ public class MediaMetadataRetriever
} catch (IOException ioEx) {
throw new IllegalArgumentException();
}
-
- try {
- if (is != null) {
- is.close();
- }
- } catch (Exception e) {}
}
/**