commit 0fe51b11ad3cebb12d1973fbc250c53d6d3cc9b6
author Hunter Knepshield <hknepshield@google.com> Thu Mar 05 16:58:20 2020 -0800
committer Hunter Knepshield <hknepshield@google.com> Fri Mar 06 03:20:10 2020 +0000
tree 4e473f136f62f155e5bc665af8282f8aead54c60
parent 24e9660ce414a5f4a1fcf9900db2be9b718de2c1

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

cmds/dumpstate/dumpstate.cpp

diff --git a/cmds/dumpstate/dumpstate.cpp b/cmds/dumpstate/dumpstate.cpp
index 3440116..e5bbb28 100644
--- a/cmds/dumpstate/dumpstate.cpp
+++ b/cmds/dumpstate/dumpstate.cpp
@@ -1728,6 +1728,8 @@
     RunDumpsys("DUMPSYS", {"netpolicy"}, CommandOptions::WithTimeout(90).Build(), SEC_TO_MSEC(10));
     RunDumpsys("DUMPSYS", {"network_management"}, CommandOptions::WithTimeout(90).Build(),
                SEC_TO_MSEC(10));
+    RunDumpsys("DUMPSYS", {"telephony.registry"}, CommandOptions::WithTimeout(90).Build(),
+               SEC_TO_MSEC(10));
     if (include_sensitive_info) {
         // Contains raw IP addresses, omit from reports on user builds.
         RunDumpsys("DUMPSYS", {"netd"}, CommandOptions::WithTimeout(90).Build(), SEC_TO_MSEC(10));