commit 539c2b12cca5e9bccd9acabc4721c876058a68d4
author Scott Kennedy <skennedy@google.com> Wed Mar 13 19:25:12 2013 -0700
committer Scott Kennedy <skennedy@google.com> Wed Mar 13 19:29:04 2013 -0700
tree 9c709875738be7f2eec02784df7c9c04ca8aa110
parent 194d427ebcfc2133bda410e0e4c399250d9a6066

Always close this cursor

Bug: 8375851

Change-Id: I6aa528091615319722ea7b101bb1cda42c97d122

src/com/android/ex/chips/RecipientAlternatesAdapter.java

diff --git a/src/com/android/ex/chips/RecipientAlternatesAdapter.java b/src/com/android/ex/chips/RecipientAlternatesAdapter.java
index 465c90e..0693df2 100644
--- a/src/com/android/ex/chips/RecipientAlternatesAdapter.java
+++ b/src/com/android/ex/chips/RecipientAlternatesAdapter.java
@@ -136,10 +136,17 @@
         final Set<String> matchesNotFound = new HashSet<String>();
         if (recipientEntries.size() < addresses.size()) {
             final List<DirectorySearchParams> paramsList;
-            Cursor directoryCursor = context.getContentResolver().query(DirectoryListQuery.URI,
-                    DirectoryListQuery.PROJECTION, null, null, null);
-            paramsList = BaseRecipientAdapter.setupOtherDirectories(context, directoryCursor,
-                    account);
+            Cursor directoryCursor = null;
+            try {
+                directoryCursor = context.getContentResolver().query(DirectoryListQuery.URI,
+                        DirectoryListQuery.PROJECTION, null, null, null);
+                paramsList = BaseRecipientAdapter.setupOtherDirectories(context, directoryCursor,
+                        account);
+            } finally {
+                if (directoryCursor != null) {
+                    directoryCursor.close();
+                }
+            }
             // Run a directory query for each unmatched recipient.
             HashSet<String> unresolvedAddresses = new HashSet<String>();
             for (String address : addresses) {