commit b1ffe935626d6760cdc4df83dbf5ef5238ce4473
author Ben Hamilton <benhamilton@google.com> Tue May 24 16:26:22 2022
committer Crashpad LUCI CQ <crashpad-scoped@luci-project-accounts.iam.gserviceaccount.com> Tue May 24 18:15:58 2022
tree e94b6619a8efd0f7985ac0527335e4823a7d3b63
parent 5f2a706f71de3204c916401f3ea084e47f689057

[ios] Fix two shutdown races

There were two shutdown races in the iOS Crashpad client:

1) MachMessageServer::Run can return either MACH_RCV_PORT_CHANGED *or*
   MACH_RCV_INVALID_NAME based on the timing of when the port is
   closed, for example:
   https://github.com/mattgallagher/CwlPreconditionTesting/blob/c21f7bab5ca8eee0a9998bbd17ca1d0eb45d4688/Sources/CwlPreconditionTesting/CwlCatchBadInstruction.swift#L131
2) The iOS crashpad::CrashHandler thread could read from its member
   variable mach_handler_running_ while another thread wrote to it

Change-Id: I696ece8575d9b88cbd0593e7c479bd4c7f863f45
Reviewed-on: https://chromium-review.googlesource.com/c/crashpad/crashpad/+/3651395
Reviewed-by: Justin Cohen <justincohen@chromium.org>
Reviewed-by: Mark Mentovai <mark@chromium.org>
Commit-Queue: Justin Cohen <justincohen@chromium.org>