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

Commit 5acca3c4 authored by Andy Hung's avatar Andy Hung Committed by Automerger Merge Worker
Browse files

Merge "MediaMetricsService: Validate logSessionId" into sc-dev am: e002ca80 

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

Change-Id: Id737902e8ac49a05351f205e1fa5b497ebdb3fe2
parents 8f1b7cc7 e002ca80

services/mediametrics/statsd_codec.cpp

+2 −0
Original line number Diff line number Diff line
@@ -34,6 +34,7 @@ 


#include "cleaner.h"

#include "MediaMetricsService.h"

#include "StringUtils.h"

#include "frameworks/proto_logging/stats/message/mediametrics_message.pb.h"

#include "iface_statsd.h"
@@ -227,6 +228,7 @@ bool statsd_codec(const std::shared_ptr<const mediametrics::Item>& item, 


    std::string sessionId;

    if (item->getString("android.media.mediacodec.log-session-id", &sessionId)) {

        sessionId = mediametrics::stringutils::sanitizeLogSessionId(sessionId);

        metrics_proto.set_log_session_id(sessionId);

    }

    AStatsEvent_writeString(event, codec.c_str());

services/mediametrics/statsd_extractor.cpp

+2 −0
Original line number Diff line number Diff line
@@ -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 @@ bool statsd_extractor(const std::shared_ptr<const mediametrics::Item>& item, 


    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);

    }

services/mediametrics/statsd_mediaparser.cpp

+2 −0
Original line number Diff line number Diff line
@@ -31,6 +31,7 @@ 
#include <statslog.h>



#include "MediaMetricsService.h"

#include "StringUtils.h"

#include "frameworks/proto_logging/stats/enums/stats/mediametrics/mediametrics.pb.h"

#include "iface_statsd.h"
@@ -80,6 +81,7 @@ bool statsd_mediaparser(const std::shared_ptr<const mediametrics::Item>& item, 


    std::string logSessionId;

    item->getString("android.media.mediaparser.logSessionId", &logSessionId);

    logSessionId = mediametrics::stringutils::sanitizeLogSessionId(logSessionId);



    int result = android::util::stats_write(android::util::MEDIAMETRICS_MEDIAPARSER_REPORTED,

                               timestamp_nanos,

services/mediametrics/statsd_recorder.cpp

+2 −0
Original line number Diff line number Diff line
@@ -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"
@@ -58,6 +59,7 @@ bool statsd_recorder(const std::shared_ptr<const mediametrics::Item>& item, 
    // string kRecorderLogSessionId = "android.media.mediarecorder.log-session-id";

    std::string log_session_id;

    if (item->getString("android.media.mediarecorder.log-session-id", &log_session_id)) {

        log_session_id = mediametrics::stringutils::sanitizeLogSessionId(log_session_id);

        metrics_proto.set_log_session_id(log_session_id);

    }

    // string kRecorderAudioMime = "android.media.mediarecorder.audio.mime";