summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAli Utku Selen <aliutku.selen@sonyericsson.com>2010-11-30 10:37:10 +0100
committerKenneth Andersson <kenneth.andersson@sonyericsson.com>2010-11-30 16:22:56 +0100
commit1c3274ca12a6931568693b897c269107f0346eab (patch)
treef16be948fd0c11f124c2e8b402b1f290c1956879
parentf0f6c54b13831b11bd7a3bd7a01dc49b7505e56e (diff)
downloadbase-1c3274ca12a6931568693b897c269107f0346eab.tar.gz
Fix crash by "adb shell pm list permissions -f|-s"
pm command makes the assumption that every permission should contain android:description and android:label attributes while they are not mandatory. If a permission does not contain these two attributes, we get "android.content.res. Resources$NotFoundException: String resource ID #0x0" followed by a NPE when using -f or -s options. With the following change, users will get "null" in output for respective fields. Change-Id: I4e7f407592fa071abdab1d979775f46ec27dc9d2
-rw-r--r--cmds/pm/src/com/android/commands/pm/Pm.java8
1 files changed, 5 insertions, 3 deletions
diff --git a/cmds/pm/src/com/android/commands/pm/Pm.java b/cmds/pm/src/com/android/commands/pm/Pm.java
index 9b8b0ac4477a..46fa1146dae3 100644
--- a/cmds/pm/src/com/android/commands/pm/Pm.java
+++ b/cmds/pm/src/com/android/commands/pm/Pm.java
@@ -341,9 +341,11 @@ public final class Pm {
if (nonLocalized != null) {
return nonLocalized.toString();
}
- Resources r = getResources(pii);
- if (r != null) {
- return r.getString(res);
+ if (res != 0) {
+ Resources r = getResources(pii);
+ if (r != null) {
+ return r.getString(res);
+ }
}
return null;
}