commit c5e057c736dd6552586b104bcd06a6c12c573e28
author Suprabh Shukla <suprabh@google.com> Mon Aug 08 16:22:44 2016 -0700
committer Suprabh Shukla <suprabh@google.com> Mon Aug 08 16:22:50 2016 -0700
tree b5f2c18976dda98e0f7c5890cbf80d7e9ef22d0f
parent c2fe7f727957900a91f1d537f2103c17eea101d6

Checking for storage space before adding user

If storage space is low, trying to create a user results in a crash.
Adding a check to abort user creation and log a warning in such a case.

Bug: 30270050
Change-Id: I121119aed340c7be337b2147c4386c9bb9f8ffdc

services/core/java/com/android/server/pm/UserManagerService.java

diff --git a/services/core/java/com/android/server/pm/UserManagerService.java b/services/core/java/com/android/server/pm/UserManagerService.java
index e489d96..bdb138e 100644
--- a/services/core/java/com/android/server/pm/UserManagerService.java
+++ b/services/core/java/com/android/server/pm/UserManagerService.java
@@ -95,7 +95,7 @@
 import com.android.server.LocalServices;
 import com.android.server.SystemService;
 import com.android.server.am.UserState;
-
+import com.android.server.storage.DeviceStorageMonitorInternal;
 import libcore.io.IoUtils;
 import libcore.util.Objects;
 
@@ -2184,6 +2184,12 @@
             Log.w(LOG_TAG, "Cannot add user. DISALLOW_ADD_USER is enabled.");
             return null;
         }
+        DeviceStorageMonitorInternal dsm = LocalServices
+                .getService(DeviceStorageMonitorInternal.class);
+        if (dsm.isMemoryLow()) {
+            Log.w(LOG_TAG, "Cannot add user. Not enough space on disk.");
+            return null;
+        }
         return createUserInternalUnchecked(name, flags, parentId);
     }