Skip to content

Commit 2c02228

Browse files
sganovAkshay Thakker
authored andcommitted
DO NOT MERGE Modify system settings: incorrect UI state
The privilege for an app to write to the system settings is protected by an app-op signature permission. App-op permissions are special: if the app-op is deny/allow we deny/allow write access; if the app-op is default holding the permission determies write access. The settings code assumes that CHANGE_NETWORK_STATE is an app op permission (system|appop) while it is a normal permission which any app gets by declaring it used in the manifest. The side effect is that the state of the toggle in the UI for write system settings will initially be in the wrong state if the app uses both WRITE_SETTINGS and CHANGE_NETWORK_STATE. However, the code in the public API an app uses to check write settings access would return the opposite since it checks the WRITE_SETTINGS permission and its app op. Hence, if an app requires write settings to start the user will see in the settings UI it has access but the app will not have access, so the app would prompt the user to allow write settings. The non-obvious fix is for the user to toggle the setting off and on to get the app op in the right state and be able to launch the app. bug:25843134 Change-Id: I3d726a66c7f9857bc7dbd5946fdbb8f340c6eb4d (cherry picked from commit 356fb2d) (cherry picked from commit 119d589)
1 parent 21e3d54 commit 2c02228

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

src/com/android/settings/applications/AppStateWriteSettingsBridge.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -34,11 +34,9 @@ public class AppStateWriteSettingsBridge extends AppStateAppOpsBridge {
3434
private static final String TAG = "AppStateWriteSettingsBridge";
3535
private static final int APP_OPS_OP_CODE = AppOpsManager.OP_WRITE_SETTINGS;
3636
private static final String PM_WRITE_SETTINGS = Manifest.permission.WRITE_SETTINGS;
37-
private static final String PM_CHANGE_NETWORK_STATE = Manifest.permission.CHANGE_NETWORK_STATE;
38-
// CHANGE_NETWORK_STATE is now merged with WRITE_SETTINGS
37+
3938
private static final String[] PM_PERMISSIONS = {
40-
PM_WRITE_SETTINGS,
41-
PM_CHANGE_NETWORK_STATE
39+
PM_WRITE_SETTINGS
4240
};
4341

4442
public AppStateWriteSettingsBridge(Context context, ApplicationsState appState, Callback

0 commit comments

Comments
 (0)