[BugFix]: Remove the android property used by GXP debug dump [RESTRICT AUTOMERGE]

"dump_gxp" needs access to gxp android properties earlier.
Granting access causes compatibility issues between P23 & P22 devices,
thus removing this property is the easiest option.

Bug: 287898138
Change-Id: I9ebebbc4f7c47cfbc8efd1d5dede080d79011736
Signed-off-by: Dinesh Yadav <dkyadav@google.com>
This commit is contained in:
Dinesh Yadav 2023-06-20 07:48:07 +00:00
parent 05961615f6
commit 3f904fffac

View file

@ -21,17 +21,17 @@
int main() { int main() {
/* TODO(b/277094681): Set default value to false around device beta milestone. */ /* TODO(b/277094681): Set default value to false around device beta milestone. */
if(::android::base::GetBoolProperty("vendor.gxp.attach_to_bugreport", true)) { /* TODO(b/288368306): Create an android property for enabling/disabling
std::string outputDir = concatenatePath(BUGREPORT_PACKING_DIR, "gxp_ssrdump"); * bugreport collection for gxp */
printf("Creating %s", outputDir.c_str()); std::string outputDir = concatenatePath(BUGREPORT_PACKING_DIR, "gxp_ssrdump");
if (mkdir(outputDir.c_str(), 0777) == -1) { printf("Creating %s", outputDir.c_str());
printf("Unable to create folder: %s\n", outputDir.c_str()); if (mkdir(outputDir.c_str(), 0777) == -1) {
return 0; printf("Unable to create folder: %s\n", outputDir.c_str());
} return 0;
dumpLogs("/data/vendor/ssrdump/coredump", outputDir.c_str(), maxGxpDebugDumps, "coredump_gxp_");
dumpLogs("/data/vendor/ssrdump", outputDir.c_str(), maxGxpDebugDumps, "crashinfo_gxp_");
} }
dumpLogs("/data/vendor/ssrdump/coredump", outputDir.c_str(), maxGxpDebugDumps, "coredump_gxp_");
dumpLogs("/data/vendor/ssrdump", outputDir.c_str(), maxGxpDebugDumps, "crashinfo_gxp_");
return 0; return 0;
} }