diff options
| author | Ali Utku Selen <aliutku.selen@sonyericsson.com> | 2010-11-30 10:37:10 +0100 |
|---|---|---|
| committer | Kenneth Andersson <kenneth.andersson@sonyericsson.com> | 2010-11-30 16:22:56 +0100 |
| commit | 1c3274ca12a6931568693b897c269107f0346eab (patch) | |
| tree | f16be948fd0c11f124c2e8b402b1f290c1956879 /cmds/pm | |
| parent | f0f6c54b13831b11bd7a3bd7a01dc49b7505e56e (diff) | |
| download | frameworks_base-1c3274ca12a6931568693b897c269107f0346eab.zip frameworks_base-1c3274ca12a6931568693b897c269107f0346eab.tar.gz frameworks_base-1c3274ca12a6931568693b897c269107f0346eab.tar.bz2 | |
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
Diffstat (limited to 'cmds/pm')
| -rw-r--r-- | cmds/pm/src/com/android/commands/pm/Pm.java | 8 |
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 9b8b0ac..46fa114 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; } |