commit 11e60d7aadbbcaca617079111407f197dd5ab113
author Andy Hung <hunga@google.com> Thu Jul 08 10:02:11 2021 -0700
committer Andy Hung <hunga@google.com> Thu Jul 08 13:30:39 2021 -0700
tree 0c21913dc101d0eee8443ef73a74cc44690fd6a1
parent 6c14e31d430315317b0f6653a75ba01ab0c4e6ae

MediaMetricsService: Validate logSessionId

Suppress malformed user supplied data (returns empty
id instead).

Test: atest AudioTrackTest#testSetLogSessionId
Test: atest AudioRecordTest#testSetLogSessionId
Test: atest MediaRecorderTest#testSetGetLogSessionId
Test: atest MediaDrmTest#testPlaybackComponent
Bug: 193164585
Change-Id: I56cab989d8223959762a5978192b0f41751ad01c

services/mediametrics/statsd_extractor.cpp

diff --git a/services/mediametrics/statsd_extractor.cpp b/services/mediametrics/statsd_extractor.cpp
index 4ac5621..bcf2e0a 100644
--- a/services/mediametrics/statsd_extractor.cpp
+++ b/services/mediametrics/statsd_extractor.cpp
@@ -32,6 +32,7 @@
 #include <statslog.h>
 
 #include "MediaMetricsService.h"
+#include "StringUtils.h"
 #include "frameworks/proto_logging/stats/message/mediametrics_message.pb.h"
 #include "iface_statsd.h"
 
@@ -85,6 +86,7 @@
 
     std::string log_session_id;
     if (item->getString("android.media.mediaextractor.logSessionId", &log_session_id)) {
+        log_session_id = mediametrics::stringutils::sanitizeLogSessionId(log_session_id);
         metrics_proto.set_log_session_id(log_session_id);
     }