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

Commit e350dec9 authored by Mitch Phillips's avatar Mitch Phillips Committed by Automerger Merge Worker
Browse files

Merge "[debuggerd] New protocol to AM to handle recoverable crashes." am: 5a2a3021 am: 0fe58790 

Original change: https://android-review.googlesource.com/c/platform/system/core/+/2485619



Change-Id: I786f869faa32d932be3fbcc168f3f66814ffa19e
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 92824d9d 0fe58790

debuggerd/crash_dump.cpp

+23 −9
Original line number Diff line number Diff line
@@ -142,7 +142,8 @@ static bool ptrace_interrupt(pid_t tid, int* received_signal) { 
  return false;

}



static bool activity_manager_notify(pid_t pid, int signal, const std::string& amfd_data) {

static bool activity_manager_notify(pid_t pid, int signal, const std::string& amfd_data,

                                    bool recoverable_gwp_asan_crash) {

  ATRACE_CALL();

  android::base::unique_fd amfd(socket_local_client(

      "/data/system/ndebugsocket", ANDROID_SOCKET_NAMESPACE_FILESYSTEM, SOCK_STREAM));
@@ -165,19 +166,32 @@ static bool activity_manager_notify(pid_t pid, int signal, const std::string& am 
    return false;

  }



  // Activity Manager protocol: binary 32-bit network-byte-order ints for the

  // pid and signal number, followed by the raw text of the dump, culminating

  // in a zero byte that marks end-of-data.

  // Activity Manager protocol:

  //  - 32-bit network-byte-order: pid

  //  - 32-bit network-byte-order: signal number

  //  - byte: recoverable_gwp_asan_crash

  //  - bytes: raw text of the dump

  //  - null terminator



  uint32_t datum = htonl(pid);

  if (!android::base::WriteFully(amfd, &datum, 4)) {

  if (!android::base::WriteFully(amfd, &datum, sizeof(datum))) {

    PLOG(ERROR) << "AM pid write failed";

    return false;

  }



  datum = htonl(signal);

  if (!android::base::WriteFully(amfd, &datum, 4)) {

    PLOG(ERROR) << "AM signal write failed";

  if (!android::base::WriteFully(amfd, &datum, sizeof(datum))) {

    PLOG(ERROR) << "AM signo write failed";

    return false;

  }



  uint8_t recoverable_gwp_asan_crash_byte = recoverable_gwp_asan_crash ? 1 : 0;

  if (!android::base::WriteFully(amfd, &recoverable_gwp_asan_crash_byte,

                                 sizeof(recoverable_gwp_asan_crash_byte))) {

    PLOG(ERROR) << "AM recoverable_gwp_asan_crash_byte write failed";

    return false;

  }



  if (!android::base::WriteFully(amfd, amfd_data.c_str(), amfd_data.size() + 1)) {

    PLOG(ERROR) << "AM data write failed";

    return false;
@@ -651,10 +665,10 @@ int main(int argc, char** argv) { 
    }

  }



  if (fatal_signal && !recoverable_gwp_asan_crash) {

  if (fatal_signal) {

    // Don't try to notify ActivityManager if it just crashed, or we might hang until timeout.

    if (thread_info[target_process].thread_name != "system_server") {

      activity_manager_notify(target_process, signo, amfd_data);

      activity_manager_notify(target_process, signo, amfd_data, recoverable_gwp_asan_crash);

    }

  }