From 3726d5247ac10578ff1677b085d2d905027613d8 Mon Sep 17 00:00:00 2001
From: Songchun Fan <schfan@google.com>
Date: Thu, 28 Apr 2022 23:26:29 +0000
Subject: [PATCH] Revert "[SettingsProvider] enforce permission on openFile"

This reverts commit 5b50cebd18661f9ec3e0ca05516918afbabdb02b.

Reason for revert: should check appops permission instead because the permission is granted upon user approval

Bug: 230791848

Change-Id: Icc86eb38372753a5f6aa8b91918ecb4b120c1fa8
---
 .../src/com/android/providers/settings/SettingsProvider.java   | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
index ecfe4850cdd1e..077337cdc8c3e 100644
--- a/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
+++ b/packages/SettingsProvider/src/com/android/providers/settings/SettingsProvider.java
@@ -818,9 +818,6 @@ public class SettingsProvider extends ContentProvider {
             getContext().enforceCallingPermission(Manifest.permission.INTERACT_ACROSS_USERS,
                     "Access files from the settings of another user");
         }
-        if (mode.contains("w")) {
-            enforceWritePermission(Manifest.permission.WRITE_SETTINGS);
-        }
         uri = ContentProvider.getUriWithoutUserId(uri);
 
         final String cacheRingtoneSetting;