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

Commit afe84f6b authored by TreeHugger Robot's avatar TreeHugger Robot Committed by Automerger Merge Worker
Browse files

Merge "GpuService: secure setUpdatableDriverPath" into rvc-dev-plus-aosp am: 42b0a106 

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

Change-Id: I6adf7a61bf8ac8dedbc123b19208e83deb95f002
parents e9d251bb 42b0a106

services/gpuservice/GpuService.cpp

+14 −2
Original line number Diff line number Diff line
@@ -63,11 +63,23 @@ void GpuService::setTargetStats(const std::string& appPackageName, const uint64_ 
}



void GpuService::setUpdatableDriverPath(const std::string& driverPath) {

    developerDriverPath = driverPath;

    IPCThreadState* ipc = IPCThreadState::self();

    const int pid = ipc->getCallingPid();

    const int uid = ipc->getCallingUid();



    // only system_server is allowed to set updatable driver path

    if (uid != AID_SYSTEM) {

        ALOGE("Permission Denial: can't set updatable driver path from pid=%d, uid=%d\n", pid, uid);

        return;

    }



    std::lock_guard<std::mutex> lock(mLock);

    mDeveloperDriverPath = driverPath;

}



std::string GpuService::getUpdatableDriverPath() {

    return developerDriverPath;

    std::lock_guard<std::mutex> lock(mLock);

    return mDeveloperDriverPath;

}



status_t GpuService::shellCommand(int /*in*/, int out, int err, std::vector<String16>& args) {

services/gpuservice/GpuService.h

+2 −1
Original line number Diff line number Diff line
@@ -75,7 +75,8 @@ private: 
     * Attributes

     */

    std::unique_ptr<GpuStats> mGpuStats;

    std::string developerDriverPath;

    std::mutex mLock;

    std::string mDeveloperDriverPath;

};



} // namespace android