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

Commit 0fe51b11 authored by Hunter Knepshield's avatar Hunter Knepshield
Browse files

Include TelephonyRegistry in connectivity bug reports. 

This appears to be a simple case of omission, and it often contains
useful debugging information that survives telephony process crashes
since it lives in the system server.

Package names have already been scrubbed from the dump (from change
Iad7be2d7), and the output contains no other PII.

Bug: 150409813
Bug: 146521742
Test: adb shell am bug-report --telephony, verify TelReg is dumped
Change-Id: I068d50f1247d4e266d90b0557c6166cabd1bcf3f
parent 24e9660c

cmds/dumpstate/dumpstate.cpp

+2 −0
Original line number Original line Diff line number Diff line
@@ -1728,6 +1728,8 @@ static void DumpstateTelephonyOnly(const std::string& calling_package) { 
    RunDumpsys("DUMPSYS", {"netpolicy"}, CommandOptions::WithTimeout(90).Build(), SEC_TO_MSEC(10));

    RunDumpsys("DUMPSYS", {"netpolicy"}, CommandOptions::WithTimeout(90).Build(), SEC_TO_MSEC(10));

    RunDumpsys("DUMPSYS", {"network_management"}, CommandOptions::WithTimeout(90).Build(),

    RunDumpsys("DUMPSYS", {"network_management"}, CommandOptions::WithTimeout(90).Build(),

               SEC_TO_MSEC(10));

               SEC_TO_MSEC(10));

    RunDumpsys("DUMPSYS", {"telephony.registry"}, CommandOptions::WithTimeout(90).Build(),

               SEC_TO_MSEC(10));

    if (include_sensitive_info) {

    if (include_sensitive_info) {

        // Contains raw IP addresses, omit from reports on user builds.

        // Contains raw IP addresses, omit from reports on user builds.

        RunDumpsys("DUMPSYS", {"netd"}, CommandOptions::WithTimeout(90).Build(), SEC_TO_MSEC(10));

        RunDumpsys("DUMPSYS", {"netd"}, CommandOptions::WithTimeout(90).Build(), SEC_TO_MSEC(10));