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

Commit e002ca80 authored by Andy Hung's avatar Andy Hung Committed by Android (Google) Code Review
Browse files

Merge "MediaMetricsService: Validate logSessionId" into sc-dev

parents 2e57b535 11e60d7a

services/mediametrics/statsd_codec.cpp

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




#include "cleaner.h"

#include "cleaner.h"

#include "MediaMetricsService.h"

#include "MediaMetricsService.h"

#include "StringUtils.h"

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

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

#include "iface_statsd.h"

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




    std::string sessionId;

    std::string sessionId;

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

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

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

        metrics_proto.set_log_session_id(sessionId);

        metrics_proto.set_log_session_id(sessionId);

    }

    }

    AStatsEvent_writeString(event, codec.c_str());

    AStatsEvent_writeString(event, codec.c_str());

services/mediametrics/statsd_extractor.cpp

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

#include <statslog.h>





#include "MediaMetricsService.h"

#include "MediaMetricsService.h"

#include "StringUtils.h"

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

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

#include "iface_statsd.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;

    std::string log_session_id;

    if (item->getString("android.media.mediaextractor.logSessionId", &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);

        metrics_proto.set_log_session_id(log_session_id);

    }

    }

services/mediametrics/statsd_mediaparser.cpp

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

#include <statslog.h>





#include "MediaMetricsService.h"

#include "MediaMetricsService.h"

#include "StringUtils.h"

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

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

#include "iface_statsd.h"

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




    std::string logSessionId;

    std::string logSessionId;

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

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

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





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

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

                               timestamp_nanos,

                               timestamp_nanos,

services/mediametrics/statsd_recorder.cpp

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

#include <statslog.h>





#include "MediaMetricsService.h"

#include "MediaMetricsService.h"

#include "StringUtils.h"

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

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

#include "iface_statsd.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";

    // string kRecorderLogSessionId = "android.media.mediarecorder.log-session-id";

    std::string log_session_id;

    std::string log_session_id;

    if (item->getString("android.media.mediarecorder.log-session-id", &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);

        metrics_proto.set_log_session_id(log_session_id);

    }

    }

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

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