commit 3429ad80f33d506992102d9dea25fe73b692f127
author Svetoslav <svetoslavganov@google.com> Thu Apr 16 16:47:03 2015 -0700
committer Svetoslav Ganov <svetoslavganov@google.com> Fri Apr 17 00:39:26 2015 +0000
tree 48cafbed087feb577148c6175173005faa270373
parent 8f25391acd402d67243b332e8b1e312addf88d3d

Remove unnecessary WRITE_SMS permission - telephony provider

Change-Id: Iba68d6f59c38e97f34d835b54802103b845a2483

AndroidManifest.xml

diff --git a/AndroidManifest.xml b/AndroidManifest.xml
index f1cd72b..43e5e69 100644
--- a/AndroidManifest.xml
+++ b/AndroidManifest.xml
@@ -43,19 +43,20 @@
                   android:multiprocess="false"
                   android:exported="true"
                   android:singleUser="true"
-                  android:readPermission="android.permission.READ_SMS"
-                  android:writePermission="android.permission.WRITE_SMS" />
+                  android:readPermission="android.permission.READ_SMS" />
 
         <!-- This is a singleton provider that is used by all users.
              A new instance is not created for each user. And the db is shared
-             as well. -->
+             as well.
+
+             Note: We do not require a write permission as it is guarded by an app op.
+              -->
         <provider android:name="MmsProvider"
                   android:authorities="mms"
                   android:multiprocess="false"
                   android:exported="true"
                   android:singleUser="true"
-                  android:readPermission="android.permission.READ_SMS"
-                  android:writePermission="android.permission.WRITE_SMS">
+                  android:readPermission="android.permission.READ_SMS">
             <grant-uri-permission android:pathPrefix="/part/" />
             <grant-uri-permission android:pathPrefix="/drm/" />
         </provider>
@@ -68,8 +69,7 @@
                   android:multiprocess="false"
                   android:exported="true"
                   android:singleUser="true"
-                  android:readPermission="android.permission.READ_SMS"
-                  android:writePermission="android.permission.WRITE_SMS" />
+                  android:readPermission="android.permission.READ_SMS" />
 
         <provider android:name="HbpcdLookupProvider"
                   android:authorities="hbpcd_lookup"