From 1306a67d7c7da6b73ef73b4ffc71f07bb78db2e9 Mon Sep 17 00:00:00 2001 From: Jeff Hamilton Date: Wed, 30 Sep 2009 01:57:10 -0700 Subject: [PATCH] Don't pass the query parameter in if the calling package is null. The provider would read the string "null" and always refuse restricted access. Change-Id: I1a592c2284ada8ab3c41bc71e510bd4ec7fd05d8 --- src/com/android/contacts/ContactsListActivity.java | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/com/android/contacts/ContactsListActivity.java b/src/com/android/contacts/ContactsListActivity.java index f164289..529f145 100644 --- a/src/com/android/contacts/ContactsListActivity.java +++ b/src/com/android/contacts/ContactsListActivity.java @@ -1562,10 +1562,14 @@ public class ContactsListActivity extends ListActivity implements mQueryHandler.setLoadingJoinSuggestions(false); String[] projection = getProjectionForQuery(); - Uri uri = getUriToQuery().buildUpon() - .appendQueryParameter(ContactsContract.REQUESTING_PACKAGE_PARAM_KEY, - getCallingPackage()) - .build(); + String callingPackage = getCallingPackage(); + Uri uri = getUriToQuery(); + if (!TextUtils.isEmpty(callingPackage)) { + uri = uri.buildUpon() + .appendQueryParameter(ContactsContract.REQUESTING_PACKAGE_PARAM_KEY, + callingPackage) + .build(); + } // Kick off the new query switch (mMode) { -- 2.11.0