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

Commit 55d7c379 authored by Ruchir Rastogi's avatar Ruchir Rastogi Committed by Automerger Merge Worker
Browse files

Merge "Fix AppBreadcrumbReported bug with isolated uid" into rvc-dev am: 62a2b602 am: f7952e93 

Change-Id: I6c00df7246ad085fd03dc3a40b4946decaf8fed3
parents 89c07301 f7952e93

cmds/statsd/src/metrics/MetricsManager.cpp

+8 −5
Original line number Diff line number Diff line
@@ -391,7 +391,10 @@ bool MetricsManager::eventSanityCheck(const LogEvent& event) { 
            VLOG("APP_BREADCRUMB_REPORTED had error when parsing the uid");

            return false;

        }

        int32_t loggerUid = event.GetUid();



        // Because the uid within the LogEvent may have been mapped from

        // isolated to host, map the loggerUid similarly before comparing.

        int32_t loggerUid = mUidMap->getHostUidOrSelf(event.GetUid());

        if (loggerUid != appHookUid && loggerUid != AID_STATSD) {

            VLOG("APP_BREADCRUMB_REPORTED has invalid uid: claimed %ld but caller is %d",

                 appHookUid, loggerUid);