Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit ef3b79c3 authored by Christopher Tate's avatar Christopher Tate Committed by The Android Automerger
Browse files

Settings provider needs to send notifications as itself 

... and not as its ultimate caller, who may be a less-privileged
application.  Fixes bug 7188309

Change-Id: Iffd37b8da84f683bf665bf3d48c0b7fbc8dd721d
parent 4b3a9344

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

+6 −1
Original line number Diff line number Diff line
@@ -344,7 +344,12 @@ public class SettingsProvider extends ContentProvider { 
        String notify = uri.getQueryParameter("notify");

        if (notify == null || "true".equals(notify)) {

            final int notifyTarget = isGlobal ? UserHandle.USER_ALL : userHandle;

            final long oldId = Binder.clearCallingIdentity();

            try {

                getContext().getContentResolver().notifyChange(uri, null, true, notifyTarget);

            } finally {

                Binder.restoreCallingIdentity(oldId);

            }

            if (LOCAL_LOGV) Log.v(TAG, "notifying for " + notifyTarget + ": " + uri);

        } else {

            if (LOCAL_LOGV) Log.v(TAG, "notification suppressed: " + uri);