code left opened files behind

verified with lsof DownloadProvider after downloading a file shows:
${proc} 338    10034   33w   REG      179,0  167634    5 /sdcard/download/fw4-1.pdf

Change-Id: I8e2412fe9a6348f5ece6f5ca3a9ebf99a4474bce

1 files changed, 11 insertions, 1 deletions

diff --git a/src/com/android/providers/downloads/DownloadThread.java b/src/com/android/providers/downloads/DownloadThread.java
index d899314b..d0a33bdf 100644
--- a/src/com/android/providers/downloads/DownloadThread.java
+++ b/src/com/android/providers/downloads/DownloadThread.java
@@ -694,8 +694,10 @@ http_request_loop:
                     FileUtils.setPermissions(filename, 0644, -1, -1);
 
                     // Sync to storage after completion
+                    FileOutputStream downloadedFileStream = null;
                     try {
-                        new FileOutputStream(filename, true).getFD().sync();
+                        downloadedFileStream = new FileOutputStream(filename, true);
+                        downloadedFileStream.getFD().sync();
                     } catch (FileNotFoundException ex) {
                         Log.w(Constants.TAG, "file " + filename + " not found: " + ex);
                     } catch (SyncFailedException ex) {
@@ -704,6 +706,14 @@ http_request_loop:
                         Log.w(Constants.TAG, "IOException trying to sync " + filename + ": " + ex);
                     } catch (RuntimeException ex) {
                         Log.w(Constants.TAG, "exception while syncing file: ", ex);
+                    } finally {
+                        if(downloadedFileStream != null) {
+                            try {
+                                downloadedFileStream.close();
+                            } catch (IOException e) {
+                                Log.w(Constants.TAG, "IOException while closing synced file: ", e);
+                            }
+                        }
                     }
                 }
             }