commit 16dd82cfdc879b7c3e51b19e54c70dbf78e8d697
author Brett Chabot <brettchabot@android.com> Thu Jun 18 17:00:48 2009 -0700
committer Brett Chabot <brettchabot@android.com> Thu Jun 18 17:05:03 2009 -0700
tree 2c100cbb5e0d086ddbb2415a9146ae213d6a4195
parent 6cdd1c8a60f56eabbdb09e4c1231975c96715388

Make SettingsProviders SecurityException messages more verbose.

packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java

diff --git a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
index 6f430c4..18a69c2 100644
--- a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
+++ b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
@@ -158,14 +158,18 @@
                 getContext().checkCallingOrSelfPermission(
                         android.Manifest.permission.WRITE_SECURE_SETTINGS) !=
                     PackageManager.PERMISSION_GRANTED) {
-                throw new SecurityException("Cannot write secure settings table");
-        
+                throw new SecurityException(
+                        String.format("Permission denial: writing to secure settings requires $1%s",
+                                android.Manifest.permission.WRITE_SECURE_SETTINGS));
+
         // TODO: Move gservices into its own provider so we don't need this nonsense.
         } else if ("gservices".equals(args.table) &&
             getContext().checkCallingOrSelfPermission(
                     android.Manifest.permission.WRITE_GSERVICES) !=
                 PackageManager.PERMISSION_GRANTED) {
-            throw new SecurityException("Cannot write gservices table");
+            throw new SecurityException(
+                    String.format("Permission denial: writing to gservices settings requires $1%s",
+                            android.Manifest.permission.WRITE_GSERVICES));
         }
     }