summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Kenny Root <kroot@google.com> 2010-08-02 12:33:40 -0700
committer Android (Google) Code Review <android-gerrit@google.com> 2010-08-02 12:33:40 -0700
commit739cdab7cb63d53440bb411fa2d558e2114b4c81 (patch)
treed51390327a6d9de2fc7ef118a403b07393ddf4a6
parent1540a7ea1afcb772b5fcf90f635153f6489eb8ed (diff)
parent930d3af75f9e9663222f4c4a1d75b326cf811e35 (diff)
Merge "Do not deference before null check" into gingerbread
-rw-r--r--core/java/android/content/pm/PackageParser.java2
-rw-r--r--packages/DefaultContainerService/src/com/android/defcontainer/DefaultContainerService.java3
2 files changed, 2 insertions, 3 deletions
diff --git a/core/java/android/content/pm/PackageParser.java b/core/java/android/content/pm/PackageParser.java
index 73c401135522..0b35d8b1c0da 100644
--- a/core/java/android/content/pm/PackageParser.java
+++ b/core/java/android/content/pm/PackageParser.java
@@ -587,7 +587,7 @@ public class PackageParser {
* location from the apk location at the given file path.
* @param packageFilePath file location of the apk
* @param flags Special parse flags
- * @return PackageLite object with package information.
+ * @return PackageLite object with package information or null on failure.
*/
public static PackageLite parsePackageLite(String packageFilePath, int flags) {
XmlResourceParser parser = null;
diff --git a/packages/DefaultContainerService/src/com/android/defcontainer/DefaultContainerService.java b/packages/DefaultContainerService/src/com/android/defcontainer/DefaultContainerService.java
index 6eaf0ccf6410..f1c6532a90fb 100644
--- a/packages/DefaultContainerService/src/com/android/defcontainer/DefaultContainerService.java
+++ b/packages/DefaultContainerService/src/com/android/defcontainer/DefaultContainerService.java
@@ -125,8 +125,6 @@ public class DefaultContainerService extends IntentService {
metrics.setToDefaults();
PackageParser.PackageLite pkg = packageParser.parsePackageLite(
archiveFilePath, 0);
- ret.packageName = pkg.packageName;
- ret.installLocation = pkg.installLocation;
// Nuke the parser reference right away and force a gc
packageParser = null;
Runtime.getRuntime().gc();
@@ -136,6 +134,7 @@ public class DefaultContainerService extends IntentService {
return ret;
}
ret.packageName = pkg.packageName;
+ ret.installLocation = pkg.installLocation;
ret.recommendedInstallLocation = recommendAppInstallLocation(pkg.installLocation, archiveFilePath, flags);
return ret;
}