From 28fe20930e4749a7d34bb438047c0647f775cf67 Mon Sep 17 00:00:00 2001
From: Bjorn Bringert <bringert@android.com>
Date: Tue, 6 Apr 2010 14:48:41 +0100
Subject: [PATCH] Guard against NPE in SearchManagerService

Bug http://b/issue?id=2570569

Change-Id: I72395f847b3dbc699c73358e14421d624f40fcf3
---
 core/java/android/app/SearchableInfo.java | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/core/java/android/app/SearchableInfo.java b/core/java/android/app/SearchableInfo.java
index 0cdf971f988de..5482f605ec8fc 100644
--- a/core/java/android/app/SearchableInfo.java
+++ b/core/java/android/app/SearchableInfo.java
@@ -258,9 +258,9 @@ public final class SearchableInfo implements Parcelable {
         try {
             theirContext = context.createPackageContext(activity.getPackageName(), 0);
         } catch (PackageManager.NameNotFoundException e) {
-            // unexpected, but we deal with this by null-checking theirContext
+            Log.e(LOG_TAG, "Package not found " + activity.getPackageName());
         } catch (java.lang.SecurityException e) {
-            // unexpected, but we deal with this by null-checking theirContext
+            Log.e(LOG_TAG, "Can't make context for " + activity.getPackageName(), e);
         }
         
         return theirContext;
@@ -552,7 +552,8 @@ public final class SearchableInfo implements Parcelable {
             final ComponentName cName)  {
         SearchableInfo result = null;
         Context activityContext = createActivityContext(context, cName);
-        
+        if (activityContext == null) return null;
+
         // in order to use the attributes mechanism, we have to walk the parser
         // forward through the file until it's reading the tag of interest.
         try {