From b1bf9cf3835ce41f3864f1056554f0180624e2c7 Mon Sep 17 00:00:00 2001 From: Amith Yamasani Date: Wed, 19 Sep 2012 10:58:32 -0700 Subject: [PATCH] Fix a crash in InstalledAppDetails Initialize mPackageInfo in onCreate Bug: 7194755 Change-Id: Ic82351591bd38c4c161bbb23c4033038d1d7fa85 --- .../applications/InstalledAppDetails.java | 23 +++++++++++-------- 1 file changed, 13 insertions(+), 10 deletions(-) diff --git a/src/com/android/settings/applications/InstalledAppDetails.java b/src/com/android/settings/applications/InstalledAppDetails.java index 2284b91a9cf..97d7b75e0bd 100644 --- a/src/com/android/settings/applications/InstalledAppDetails.java +++ b/src/com/android/settings/applications/InstalledAppDetails.java @@ -564,6 +564,16 @@ public class InstalledAppDetails extends Fragment } } mAppEntry = mState.getEntry(packageName); + // Get application info again to refresh changed properties of application + try { + mPackageInfo = mPm.getPackageInfo(mAppEntry.info.packageName, + PackageManager.GET_DISABLED_COMPONENTS | + PackageManager.GET_UNINSTALLED_PACKAGES | + PackageManager.GET_SIGNATURES); + } catch (NameNotFoundException e) { + Log.e(TAG, "Exception when retrieving package:" + mAppEntry.info.packageName, e); + } + return packageName; } @@ -576,18 +586,11 @@ public class InstalledAppDetails extends Fragment if (mAppEntry == null) { return false; // onCreate must have failed, make sure to exit } - - // Get application info again to refresh changed properties of application - try { - mPackageInfo = mPm.getPackageInfo(mAppEntry.info.packageName, - PackageManager.GET_DISABLED_COMPONENTS | - PackageManager.GET_UNINSTALLED_PACKAGES | - PackageManager.GET_SIGNATURES); - } catch (NameNotFoundException e) { - Log.e(TAG, "Exception when retrieving package:" + mAppEntry.info.packageName, e); + + if (mPackageInfo == null) { return false; // onCreate must have failed, make sure to exit } - + // Get list of preferred activities List prefActList = new ArrayList();