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

Commit 14ab967e authored by Svet Ganov's avatar Svet Ganov
Browse files

Don't record audio if UID is idle - core framework 

If a UID is in an idle state we don't allow recording to protect
user's privacy. If the UID is in an idle state we allow recording
but report empty data (all zeros in the byte array) and once
the process goes in an active state we report the real mic data.
This avoids the race between the app being notified aboout its
lifecycle and the audio system being notified about the state
of a UID.

Test: Added - AudioRecordTest#testRecordNoDataForIdleUids
      Passing - cts-tradefed run cts-dev -m CtsMediaTestCases
                        -t android.media.cts.AudioRecordTest

bug:63938985
Change-Id: I15264c5c4b47813ca60280bce30b22b8b1f87eab
parent a98bd998

core/java/android/app/IActivityManager.aidl

+3 −3
Original line number Diff line number Diff line
@@ -86,6 +86,9 @@ interface IActivityManager { 
    // the ones in frameworks/native/libs/binder/include/binder/IActivityManager.h

    // =============== Beginning of transactions used on native side as well ======================

    ParcelFileDescriptor openContentUri(in String uriString);

    void registerUidObserver(in IUidObserver observer, int which, int cutpoint,

            String callingPackage);

    void unregisterUidObserver(in IUidObserver observer);

    // =============== End of transactions used on native side as well ============================



    // Special low-level communication with activity manager.
@@ -478,9 +481,6 @@ interface IActivityManager { 
     */

    void keyguardGoingAway(int flags);

    int getUidProcessState(int uid, in String callingPackage);

    void registerUidObserver(in IUidObserver observer, int which, int cutpoint,

            String callingPackage);

    void unregisterUidObserver(in IUidObserver observer);

    boolean isAssistDataAllowedOnCurrentActivity();

    boolean showAssistFromActivity(in IBinder token, in Bundle args);

    boolean isRootVoiceInteraction(in IBinder token);

core/java/android/app/IUidObserver.aidl

+20 −9
Original line number Diff line number Diff line
@@ -18,15 +18,14 @@ package android.app; 


/** {@hide} */

oneway interface IUidObserver {

    /**

     * General report of a state change of an uid.

     *

     * @param uid The uid for which the state change is being reported.

     * @param procState The updated process state for the uid.

     * @param procStateSeq The sequence no. associated with process state change of the uid,

     *                     see UidRecord.procStateSeq for details.

     */

    void onUidStateChanged(int uid, int procState, long procStateSeq);

    // WARNING: when these transactions are updated, check if they are any callers on the native

    // side. If so, make sure they are using the correct transaction ids and arguments.

    // If a transaction which will also be used on the native side is being inserted, add it to

    // below block of transactions.



    // Since these transactions are also called from native code, these must be kept in sync with

    // the ones in frameworks/native/include/binder/IActivityManager.h

    // =============== Beginning of transactions used on native side as well ======================



    /**

     * Report that there are no longer any processes running for a uid.
@@ -44,6 +43,18 @@ oneway interface IUidObserver { 
     */

    void onUidIdle(int uid, boolean disabled);



    // =============== End of transactions used on native side as well ============================



    /**

     * General report of a state change of an uid.

     *

     * @param uid The uid for which the state change is being reported.

     * @param procState The updated process state for the uid.

     * @param procStateSeq The sequence no. associated with process state change of the uid,

     *                     see UidRecord.procStateSeq for details.

     */

    void onUidStateChanged(int uid, int procState, long procStateSeq);



    /**

     * Report when the cached state of a uid has changed.

     * If true, a uid has become cached -- that is, it has some active processes that are

data/etc/platform.xml

+1 −0
Original line number Diff line number Diff line
@@ -150,6 +150,7 @@ 
    <assign-permission name="android.permission.WAKE_LOCK" uid="audioserver" />

    <assign-permission name="android.permission.UPDATE_DEVICE_STATS" uid="audioserver" />

    <assign-permission name="android.permission.UPDATE_APP_OPS_STATS" uid="audioserver" />

    <assign-permission name="android.permission.PACKAGE_USAGE_STATS" uid="audioserver" />



    <assign-permission name="android.permission.MODIFY_AUDIO_SETTINGS" uid="cameraserver" />

    <assign-permission name="android.permission.ACCESS_SURFACE_FLINGER" uid="cameraserver" />