Prevent recursive panics from happening

[nick-mobilecoin]

Previously if a panic occured while processing a panic the logic would
get stuck in a loop trying to handle the panic.
Now if a panic happens while processing a panic, a hard abort is
immediately executed.

[nick-mobilecoin]

Current dependencies on/for this PR:

• main
  • PR Update CI for SGX SDK action update #14
    • PR Move panic to panic/abort #13
      • PR Add untrusted side of stderr from enclave #15
        • PR Add mc-sgx-io #22
          • PR Add mc-sgx-panic-log #24
            • PR Add mc-sgx-panic crate for common panic handling #30
              • PR Add panicking::panic_count to mc-sgx-panic #31
                • PR Update CODEOWNERS file #32
                  • PR Add panicking::panicking() function #33
                  • PR Add public function thread::panicking() #34
                  • PR Add mc-sgx-sync crate #35
                  • PR Add poison.rs to mc-sgx-sync crate #36
                  • PR Add sys module to mc-sgx-sync #39
                  • PR Add mutex.rs to mc-sgx-sync crate #40
                  • PR Use a static buffer for panic messages #41
                  • PR Add mc-sgx-sync::sys::locks::Condvar #42
                  • PR Add Condvar to mc-sgx-sync crate #43
                  • PR Add mc-sgx-sync::sys::locks::RwLock #44
                  • PR Add RwLock to mc-sgx-sync #45
                  • PR Add example usage for mc-sgx-sync and std::sync #46
                  • PR Merge panic-abort and panic-log crates into mc-sgx-panic #48
                  • PR Prevent recursive panics from happening #50 👈
                  • PR Remove unneeded traits on sys types #51
        • PR Rename mc-sgx-io to mc-sgx-io-untrusted #21

This comment was auto-generated by Graphite.

[codecov-commenter]

Codecov Report

Merging #50 (d97b920) into nick/panic-log-aborts (cdc44ad) will not change coverage.
The diff coverage is n/a.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@                  Coverage Diff                   @@
##           nick/panic-log-aborts      #50   +/-   ##
======================================================
  Coverage                  92.87%   92.87%           
======================================================
  Files                          9        9           
  Lines                        323      323           
======================================================
  Hits                         300      300           
  Misses                        23       23           

Impacted Files	Coverage Δ
panic/src/lib.rs	100.00% <ø> (ø)
panic/sys/src/lib.rs	100.00% <ø> (ø)
panic/sys/src/panicking.rs	97.56% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[jcape]

This is good, but I think we're probably going to need GLOBAL_PANIC_COUNT: AtomicUsize in order to properly handle mutex poisoning.

[nick-mobilecoin]

This is good, but I think we're probably going to need GLOBAL_PANIC_COUNT: AtomicUsize in order to properly handle mutex poisoning.

Captured this in #53