commit 5d72a8dda22cdb0c8e1ded1ca5e5122b0c25bcc7
author android-build SharedAccount <android-build@sekiwake.mtv.corp.google.com> Thu Feb 11 17:09:16 2010 -0800
committer Sriram Raman <sriramkraman@google.com> Tue Feb 16 15:45:42 2010 -0500
tree d8c7b6971d3394322801d02d7e893d7f1dcbf218
parent 6b178e472266052b4a9b28b1e4a5cef07f1ebf8b

cherry pick d60e29009c1049a9776e6630e9489e1ef3f83491

Revert "Add null checks when scanning a package."
This reverts commit 9e7ac3ba8405f22c11629091449ff35a06b72055.

services/java/com/android/server/PackageManagerService.java

diff --git a/services/java/com/android/server/PackageManagerService.java b/services/java/com/android/server/PackageManagerService.java
index 6df9f2b..86504a0 100644
--- a/services/java/com/android/server/PackageManagerService.java
+++ b/services/java/com/android/server/PackageManagerService.java
@@ -1977,12 +1977,6 @@
             }
             PackageParser.Package pkg = scanPackageLI(file, file, resFile,
                     flags|PackageParser.PARSE_MUST_BE_APK, scanMode);
-            // Don't mess around with apps in system partition.
-            if (pkg == null && (flags & PackageParser.PARSE_IS_SYSTEM) == 0) {
-                // Delete the apk
-                Log.w(TAG, "Cleaning up failed install of " + file);
-                file.delete();
-            }
         }
     }
 
@@ -2177,13 +2171,6 @@
         File scanFile, File destCodeFile, File destResourceFile,
         PackageParser.Package pkg, int parseFlags, int scanMode) {
 
-        if (scanFile == null || destCodeFile == null ||
-                destResourceFile == null) {
-            // Bail out. The resource and code paths haven't been set.
-            Log.w(TAG, " Code and resource paths haven't been set correctly");
-            mLastScanError = PackageManager.INSTALL_FAILED_INVALID_APK;
-            return null;
-        }
         mScanningPath = scanFile;
         if (pkg == null) {
             mLastScanError = PackageManager.INSTALL_PARSE_FAILED_BAD_PACKAGE_NAME;