Merge pull request #33 from brarcher/fix-leak

Fix leak
brarcher · Sep 3, 2016 · cbd642a · cbd642a
2 parents 94643d7 + f580813
commit cbd642a
Showing 1 changed file with 10 additions and 3 deletions.
diff --git a/app/src/main/java/protect/babysleepsounds/LoopingAudioPlayer.java b/app/src/main/java/protect/babysleepsounds/LoopingAudioPlayer.java
@@ -97,7 +97,6 @@ public void run()
 
                 while(Thread.currentThread().isInterrupted() == false)
                 {
-                    Log.d(TAG, "Restarting audio file");
                     is = new FileInputStream(_wavFile);
                     int read;
 
@@ -114,6 +113,14 @@ public void run()
                     }
 
                     // File completed playback, start again
+                    try
+                    {
+                        is.close();
+                    }
+                    catch(IOException e)
+                    {
+                        // Nothing to do, we are finished with the file anyway
+                    }
                 }
             }
             catch(IOException e)
@@ -122,8 +129,6 @@ public void run()
             }
             finally
             {
-                audioTrack.release();
-
                 try
                 {
                     if(is != null)
@@ -135,6 +140,8 @@ public void run()
                 {
                     Log.d(TAG, "Failed to close file", e);
                 }
+
+                audioTrack.release();
             }
 
             Log.i(TAG, "Finished playback");