commit fcfcae34bd2d74e71ee218cfd7aa3567d7c3fc26
author Jay Shrauner <shrauner@google.com> Mon Nov 24 10:23:36 2014 -0800
committer Jay Shrauner <shrauner@google.com> Mon Nov 24 10:55:33 2014 -0800
tree f507cb7169abf1bf25854a3e43b3fb2c46b109b0
parent 29dfdb59fd852bc07e97f1734257bfa96775ee5c

Don't crash when sent malformed intent

If lookupUri isn't passed in abort the intent request without crashing.

Bug:18506352
Change-Id: I128a3e4e84ec35d48d139d2cc42da38cd761ff42

src/com/android/contacts/quickcontact/QuickContactActivity.java

diff --git a/src/com/android/contacts/quickcontact/QuickContactActivity.java b/src/com/android/contacts/quickcontact/QuickContactActivity.java
index a03b514..b3b6b54 100644
--- a/src/com/android/contacts/quickcontact/QuickContactActivity.java
+++ b/src/com/android/contacts/quickcontact/QuickContactActivity.java
@@ -145,7 +145,6 @@
 import com.android.contacts.widget.MultiShrinkScroller;
 import com.android.contacts.widget.MultiShrinkScroller.MultiShrinkScrollerListener;
 import com.android.contacts.widget.QuickContactImageView;
-import com.google.common.base.Preconditions;
 import com.google.common.collect.Lists;
 
 import java.util.ArrayList;
@@ -790,7 +789,11 @@
                 QuickContact.MODE_LARGE);
         final Uri oldLookupUri = mLookupUri;
 
-        mLookupUri = Preconditions.checkNotNull(lookupUri, "missing lookupUri");
+        if (lookupUri == null) {
+            finish();
+            return;
+        }
+        mLookupUri = lookupUri;
         mExcludeMimes = intent.getStringArrayExtra(QuickContact.EXTRA_EXCLUDE_MIMES);
         if (oldLookupUri == null) {
             mContactLoader = (ContactLoader) getLoaderManager().initLoader(