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

Commit 43e821f0 authored by Hridya Valsaraju's avatar Hridya Valsaraju Committed by Automerger Merge Worker
Browse files

Merge "Use ro.product.enforce_debugfs_restrictions to mount/unmount debugfs" am: 369abe01 

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/1663449

Change-Id: I98833cd3c27bff6125f1dc56f89fff80947c1947
parents c715e51a 369abe01

cmds/dumpstate/dumpstate.cpp

+10 −8
Original line number Diff line number Diff line
@@ -2174,14 +2174,13 @@ void Dumpstate::DumpstateBoard(int out_fd) { 
    }



    /*

     * mount debugfs for non-user builds which launch with S and unmount it

     * after invoking dumpstateBoard_* methods. This is to enable debug builds

     * to not have debugfs mounted during runtime. It will also ensure that

     * debugfs is only accessed by the dumpstate HAL.

     * mount debugfs for non-user builds with ro.product.enforce_debugfs_restrictions

     * set to true and unmount it after invoking dumpstateBoard_* methods.

     * This is to enable debug builds to not have debugfs mounted during runtime.

     * It will also ensure that debugfs is only accessed by the dumpstate HAL.

     */

    auto api_level = android::base::GetIntProperty("ro.product.first_api_level", 0);

    bool mount_debugfs = !PropertiesHelper::IsUserBuild() && api_level >= 31;



    auto mount_debugfs =

        android::base::GetBoolProperty("ro.product.enforce_debugfs_restrictions", false);

    if (mount_debugfs) {

        RunCommand("mount debugfs", {"mount", "-t", "debugfs", "debugfs", "/sys/kernel/debug"},

                   AS_ROOT_20);
@@ -2289,6 +2288,9 @@ void Dumpstate::DumpstateBoard(int out_fd) { 
    }



    if (mount_debugfs) {

        auto keep_debugfs_mounted =

            android::base::GetProperty("persist.dbg.keep_debugfs_mounted", "");

        if (keep_debugfs_mounted.empty())

            RunCommand("unmount debugfs", {"umount", "/sys/kernel/debug"}, AS_ROOT_20);

    }