From bd6309d85d46cd0e9dd5e804cd06ac18a4496410 Mon Sep 17 00:00:00 2001
From: Jeremy Erickson <jerickso@cs.unc.edu>
Date: Thu, 5 Dec 2013 08:56:28 -0500
Subject: [PATCH] Fix infinite recursion bug and update text to reflect that
 deleted files are found

---
 AndroidManifest.xml                                    |  4 ++--
 res/values/strings.xml                                 |  4 ++--
 src/com/gmail/jerickson314/sdscanner/ScanFragment.java | 10 +++++++---
 3 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index 81eeae4..7b6ed94 100644
--- a/AndroidManifest.xml
+++ b/AndroidManifest.xml
@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="utf-8"?>
 <manifest xmlns:android="http://schemas.android.com/apk/res/android"
       package="com.gmail.jerickson314.sdscanner"
-      android:versionCode="5"
-      android:versionName="1.4">
+      android:versionCode="6"
+      android:versionName="1.5">
     <uses-sdk android:minSdkVersion="14" android:targetSdkVersion="19" />  
     <uses-permission android:name="android.permission.READ_EXTERNAL_STORAGE" />
     <application android:label="@string/app_name" android:icon="@drawable/ic_launcher">
diff --git a/res/values/strings.xml b/res/values/strings.xml
index 85cf8da..37acadf 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -5,8 +5,8 @@
     <string name="button_start">Start Rescan</string>
     <string name="database_proc">Examined</string>
     <string name="delete_proc">Removed reference to</string>
-    <string name="db_label">Will also check existing media database for updated files.</string>
-    <string name="db_error_failure">Encountered error reading media database, and might miss updated files or rescan up-to-date files.</string>
+    <string name="db_label">Will also check existing media database for updated or deleted files.</string>
+    <string name="db_error_failure">Encountered error reading media database, and might miss updated or deleted files or rescan up-to-date files.</string>
     <string name="db_error_recovered">Encountered error reading media database, but recovered.</string>
     <string name="db_error_retrying">Encountered error reading media database.  Retrying in 1 second...</string>
     <string name="final_proc">Processed</string>
diff --git a/src/com/gmail/jerickson314/sdscanner/ScanFragment.java b/src/com/gmail/jerickson314/sdscanner/ScanFragment.java
index 1dd4894..a6ac6aa 100644
--- a/src/com/gmail/jerickson314/sdscanner/ScanFragment.java
+++ b/src/com/gmail/jerickson314/sdscanner/ScanFragment.java
@@ -234,16 +234,20 @@ public int getId() {
 
         private void recursiveAddFiles(File file)
                 throws IOException {
+            if (mFilesToProcess.contains(file)) {
+                // Avoid infinite recursion caused by symlinks.
+                return;
+            }
+            mFilesToProcess.add(file);
             if (file.isDirectory()) {
                 boolean nomedia = new File(file, ".nomedia").exists();
                 // Only recurse downward if not blocked by nomedia.
                 if (!nomedia) {
                     for (File nextFile : file.listFiles()) {
-                        recursiveAddFiles(nextFile);
+                        recursiveAddFiles(nextFile.getCanonicalFile());
                     }
                 }
             }
-            mFilesToProcess.add(file.getCanonicalFile());
         }
 
         protected void dbOneTry() {
@@ -303,7 +307,7 @@ else if (currentItem % reportFreq == 0) {
         @Override
         protected Void doInBackground(File... files) {
             try {
-                recursiveAddFiles(files[0]);
+                recursiveAddFiles(files[0].getCanonicalFile());
             }
             catch (IOException Ex) {
                 // Do nothing.