diff options
| author | 2024-05-17 17:29:53 +0000 | |
|---|---|---|
| committer | 2024-05-17 17:29:53 +0000 | |
| commit | 3ec82a163f63b82c602e498b54f57c8fa53fe769 (patch) | |
| tree | ba2a3de1752958faf4b0fd6f3385b4d4a4570957 | |
| parent | b1c7fe98125fbf587202270713bef41f6a39d957 (diff) | |
| parent | b19525912f9d4f92be4062d2ab7f3ed3b1abf776 (diff) | |
Merge "Fix incorrect gid config leads to process/system cannot start." into main am: b19525912f
Original change: https://android-review.googlesource.com/c/platform/frameworks/base/+/3002669
Change-Id: I4044fae923c62fed34051c31159bcfb04df3bf11
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
| -rw-r--r-- | services/core/java/com/android/server/SystemConfig.java | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/services/core/java/com/android/server/SystemConfig.java b/services/core/java/com/android/server/SystemConfig.java index 79620cff2b24..1c1190fde049 100644 --- a/services/core/java/com/android/server/SystemConfig.java +++ b/services/core/java/com/android/server/SystemConfig.java @@ -1638,7 +1638,13 @@ public class SystemConfig { String gidStr = parser.getAttributeValue(null, "gid"); if (gidStr != null) { int gid = Process.getGidForName(gidStr); - perm.gids = appendInt(perm.gids, gid); + if (gid != -1) { + perm.gids = appendInt(perm.gids, gid); + } else { + Slog.w(TAG, "<group> with unknown gid \"" + + gidStr + " for permission " + name + " in " + + parser.getPositionDescription()); + } } else { Slog.w(TAG, "<group> without gid at " + parser.getPositionDescription()); |