Auto merge of #12621 - jdm:sigabrt, r=emilio

Attach more signals to the backtrace signal handler.

These changes should enable meaningful backtraces in more unusual situations where the program would otherwise exit almost silently.

---
<!-- Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: -->
- [X] `./mach build -d` does not report any errors
- [X] `./mach test-tidy` does not report any errors
- [X] These changes fix #12618 (github issue number if applicable).
- [X] These changes do not require tests because we can't verify the terminal output of the program in erroneous cases

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/12621)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2016-07-28 12:39:03 -05:00 committed by GitHub
commit 25f93b699f
5 changed files with 8 additions and 5 deletions

View file

@ -70,7 +70,10 @@ fn install_crash_handler() {
}
}
signal!(Sig::SEGV, handler);
signal!(Sig::SEGV, handler); // handle segfaults
signal!(Sig::ILL, handler); // handle stack overflow and unsupported CPUs
signal!(Sig::IOT, handler); // handle double panics
signal!(Sig::BUS, handler); // handle invalid memory access
}
#[cfg(target_os = "android")]

View file

@ -1,6 +1,6 @@
[readPixelsBadArgs.html]
type: testharness
expected: CRASH
expected: TIMEOUT
[WebGL test #0: testReadPixels]
expected: FAIL

View file

@ -1,6 +1,6 @@
[read-pixels-pack-alignment.html]
type: testharness
expected: CRASH
expected: TIMEOUT
[WebGL test #3: successfullyParsed should be true (of type boolean). Was undefined (of type undefined).]
expected: FAIL

View file

@ -1,3 +1,3 @@
[draw-arrays-out-of-bounds.html]
type: testharness
expected: CRASH
expected: TIMEOUT

View file

@ -1,3 +1,3 @@
[draw-elements-out-of-bounds.html]
type: testharness
expected: CRASH
expected: TIMEOUT