diff options
author | 2015-06-25 08:33:49 +0000 | |
---|---|---|
committer | 2015-06-25 08:33:50 +0000 | |
commit | fdb70dc4346f801673aa700f3fca2b8cc77b809c (patch) | |
tree | 62592a827185a72cc4a59d21f62359870889db2c | |
parent | a78c0e0b9aff70d03101a25b47024f6d02cb93e2 (diff) | |
parent | e1856e482933b24e66f0f432ac0cb0157787e9ce (diff) |
Merge "Fix filedescriptor leakage in MediaMetadataRetriever"
-rw-r--r-- | media/java/android/media/MediaMetadataRetriever.java | 10 |
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) {} } /** |