summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Vova Sharaienko <sharaienko@google.com> 2022-12-09 02:40:35 +0000
committer Vova Sharaienko <sharaienko@google.com> 2022-12-09 20:18:11 +0000
commit5764fc1c6185486ddc99e16f2c2ed9f67f2d9770 (patch)
tree4eb94592fca37e2afdd7971074f08bdb9753d4f9
parent13ba5335585d7aa7dc774667d2c0992db7de819c (diff)
StatsAIDL: eliminated extra reverseDomainName string copy creation
Bug: 212755214 Test: aidl_stats_client Change-Id: I93c434115c75134ed1494b521df8a209e58e2dfe
-rw-r--r--services/stats/StatsAidl.cpp6
1 files changed, 3 insertions, 3 deletions
diff --git a/services/stats/StatsAidl.cpp b/services/stats/StatsAidl.cpp
index a3b68f1dab..0f1a0d4102 100644
--- a/services/stats/StatsAidl.cpp
+++ b/services/stats/StatsAidl.cpp
@@ -30,14 +30,14 @@ namespace stats {
StatsHal::StatsHal() {}
ndk::ScopedAStatus StatsHal::reportVendorAtom(const VendorAtom& vendorAtom) {
- std::string reverseDomainName = (std::string) vendorAtom.reverseDomainName;
if (vendorAtom.atomId < 100000 || vendorAtom.atomId >= 200000) {
ALOGE("Atom ID %ld is not a valid vendor atom ID", (long) vendorAtom.atomId);
return ndk::ScopedAStatus::fromServiceSpecificErrorWithMessage(
-1, "Not a valid vendor atom ID");
}
- if (reverseDomainName.length() > 50) {
- ALOGE("Vendor atom reverse domain name %s is too long.", reverseDomainName.c_str());
+ if (vendorAtom.reverseDomainName.length() > 50) {
+ ALOGE("Vendor atom reverse domain name %s is too long.",
+ vendorAtom.reverseDomainName.c_str());
return ndk::ScopedAStatus::fromServiceSpecificErrorWithMessage(
-1, "Vendor atom reverse domain name is too long");
}