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

Commit 87602ee3 authored by Jeff Sharkey's avatar Jeff Sharkey Committed by Automerger Merge Worker
Browse files

Merge "Add dropbox entries as files to dumpstate ZIP." am: 7572f7bb am: 643a9181 am: a1e15ba5 

Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/2422939



Change-Id: Iee2d6748fe0d6312f1a570fccbad7a6f0e01f502
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents c3cea29f a1e15ba5

cmds/dumpstate/dumpstate.cpp

+15 −0
Original line number Diff line number Diff line
@@ -186,6 +186,7 @@ void add_mountinfo(); 
#define SYSTEM_TRACE_SNAPSHOT "/data/misc/perfetto-traces/bugreport/systrace.pftrace"

#define CGROUPFS_DIR "/sys/fs/cgroup"

#define SDK_EXT_INFO "/apex/com.android.sdkext/bin/derive_sdk"

#define DROPBOX_DIR "/data/system/dropbox"



// TODO(narayan): Since this information has to be kept in sync

// with tombstoned, we should just put it in a common header.
@@ -526,6 +527,15 @@ static bool skip_not_stat(const char *path) { 
    return strcmp(path + len - sizeof(stat) + 1, stat); /* .../stat? */

}



static bool skip_wtf_strictmode(const char *path) {

    if (strstr(path, "_wtf")) {

        return true;

    } else if (strstr(path, "_strictmode")) {

        return true;

    }

    return false;

}



static bool skip_none(const char* path __attribute__((unused))) {

    return false;

}
@@ -1895,6 +1905,11 @@ Dumpstate::RunStatus Dumpstate::DumpstateDefaultAfterCritical() { 
    DumpIpTablesAsRoot();

    DumpDynamicPartitionInfo();

    ds.AddDir(OTA_METADATA_DIR, true);

    if (!PropertiesHelper::IsUserBuild()) {

        // Include dropbox entry files inside ZIP, but exclude

        // noisy WTF and StrictMode entries

        dump_files("", DROPBOX_DIR, skip_wtf_strictmode, _add_file_from_fd);

    }



    // Capture any IPSec policies in play. No keys are exposed here.

    RunCommand("IP XFRM POLICY", {"ip", "xfrm", "policy"}, CommandOptions::WithTimeout(10).Build());

cmds/dumpstate/tests/dumpstate_test.cpp

+2 −1
Original line number Diff line number Diff line
@@ -1051,7 +1051,8 @@ class ZippedBugReportStreamTest : public DumpstateBaseTest { 
};



// Generate a quick LimitedOnly report redirected to a file, open it and verify entry exist.

TEST_F(ZippedBugReportStreamTest, StreamLimitedOnlyReport) {

// TODO: broken test tracked in b/249983726

TEST_F(ZippedBugReportStreamTest, DISABLED_StreamLimitedOnlyReport) {

    std::string out_path = kTestDataPath + "StreamLimitedOnlyReportOut.zip";

    android::base::unique_fd out_fd;

    CreateFd(out_path, &out_fd);