commit 6c36c788dc200c063f4fe8be578d0b7ed4f63e6c
author Greg Kaiser <gkaiser@google.com> Fri Oct 18 06:27:56 2019 -0700
committer Greg Kaiser <gkaiser@google.com> Fri Oct 18 06:27:56 2019 -0700
tree f24d6d0d1d98144bfd9ecaa60a9852f0cfa4bc3a
parent 49706acbd877e3d6e083442237482f682832e416

MediaMetrics: Don't read from move()d variable

'pkg' was std::move()d a few lines before, so we avoid reading
from it for our logging statement.

Test: TreeHugger
Change-Id: Idd25360ee0b1fbfee3ae63fb8f6c06f95d302792

services/mediaanalytics/MediaAnalyticsService.cpp

diff --git a/services/mediaanalytics/MediaAnalyticsService.cpp b/services/mediaanalytics/MediaAnalyticsService.cpp
index 42afc6b..f0cb9c1 100644
--- a/services/mediaanalytics/MediaAnalyticsService.cpp
+++ b/services/mediaanalytics/MediaAnalyticsService.cpp
@@ -567,7 +567,7 @@
         mapping.versionCode = versionCode;
         mapping.expiration = now + PKG_EXPIRATION_NS;
         ALOGV("%s: adding uid %d pkg '%s' expiration: %lld",
-                __func__, uid, pkg.c_str(), (long long)mapping.expiration);
+                __func__, uid, mapping.pkg.c_str(), (long long)mapping.expiration);
         mPkgMappings[uid] = mapping;
     }