From fbd740c3ca5a55bc18a2a73e4b4255fdb47c506f Mon Sep 17 00:00:00 2001
From: Junda Liu <junda@google.com>
Date: Thu, 18 Jun 2015 16:29:35 -0700
Subject: [PATCH] Add comments to carrierconfigmanager for required permission.

Bug: b/20826627
Change-Id: Ia8a42f4792429e56108a423f27c3f843f6b0f958
---
 telephony/java/android/telephony/CarrierConfigManager.java | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/telephony/java/android/telephony/CarrierConfigManager.java b/telephony/java/android/telephony/CarrierConfigManager.java
index bcfee30f3ffe..eeda3be8c0b6 100644
--- a/telephony/java/android/telephony/CarrierConfigManager.java
+++ b/telephony/java/android/telephony/CarrierConfigManager.java
@@ -392,6 +392,9 @@ public class CarrierConfigManager {
      * specific SIM card. If an invalid subId is used, the returned config will contain default
      * values.
      *
+     * <p>Requires Permission:
+     * {@link android.Manifest.permission#READ_PHONE_STATE READ_PHONE_STATE}
+     *
      * @param subId the subscription ID, normally obtained from {@link SubscriptionManager}.
      * @return A {@link PersistableBundle} containing the config for the given subId, or default
      *         values for an invalid subId.
@@ -413,6 +416,9 @@ public class CarrierConfigManager {
     /**
      * Gets the configuration values for the default subscription.
      *
+     * <p>Requires Permission:
+     * {@link android.Manifest.permission#READ_PHONE_STATE READ_PHONE_STATE}
+     *
      * @see #getConfigForSubId
      */
     @Nullable
@@ -427,6 +433,8 @@ public class CarrierConfigManager {
      * This should be called by a carrier service app if it wants to update config at an arbitrary
      * moment.
      * </p>
+     * <p>Requires that the calling app has carrier privileges.
+     * @see #hasCarrierPrivileges
      * <p>
      * This method returns before the reload has completed, and
      * {@link android.service.carrier.CarrierService#onLoadConfig} will be called from an
-- 
2.11.0