diff options
| author | David Gibson <david@gibson.dropbear.id.au> | 2007-12-07 15:37:45 +1100 |
|---|---|---|
| committer | Jon Loeliger <jdl@freescale.com> | 2007-12-07 07:45:44 -0600 |
| commit | 0d9d10184b03e352da144fb7e640016bf98fd615 (patch) | |
| tree | 889e4379636280669c7fe2caa0e04ae0d1f3768a | |
| parent | 3476f9608bd22803c64d5a4ede118784debd0b18 (diff) | |
| download | dtc-0d9d10184b03e352da144fb7e640016bf98fd615.tar.gz | |
dtc: Make dtc-checfails.sh script catch deaths-by-signal
Since commit 5ba0086bfd0fa6ab25f7ce1870417301a26c104f, the
dtc-checkfails.sh script does not check the return code from dtc.
That's reasonable, since depending on the checks we're testing, dtc
could either complete succesfully or return an error.
However, it's never right for dtc to SEGV or otherwise be killed by a
signal. So the script should catch that, and fail the testcase if it
happens. This patch implements this behaviour.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
| -rwxr-xr-x | tests/dtc-checkfails.sh | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/tests/dtc-checkfails.sh b/tests/dtc-checkfails.sh index cca4119..16a93e8 100755 --- a/tests/dtc-checkfails.sh +++ b/tests/dtc-checkfails.sh @@ -17,6 +17,10 @@ rm -f $TMPFILE $LOG verbose_run_log "$LOG" "$DTC" -o /dev/null "$@" ret="$?" +if [ "$ret" -gt 127 ]; then + FAIL "dtc killed by signal (ret=$ret)" +fi + for c in $CHECKS; do if ! grep -E "^(ERROR)|(Warning) \($c\):" $LOG > /dev/null; then FAIL "Failed to trigger check \"%c\"" |